Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ESIP SEM-Harmo Cryohackathon sessions Feb - April 2021 #1160

Closed
kaiiam opened this issue May 12, 2021 · 7 comments
Closed

ESIP SEM-Harmo Cryohackathon sessions Feb - April 2021 #1160

kaiiam opened this issue May 12, 2021 · 7 comments

Comments

@kaiiam
Copy link
Contributor

kaiiam commented May 12, 2021

As part of the ongoing work of the ESIP Semantic harmonization cluster's Global Cryosphere Watch (GCW) cryohackaton work over the period of February - April 2021, we @pbuttigieg @rduerr @brandonnodnarb and others worked on the following term set shown in the second sheet of this ESIP_Cryo_hackathon_robot_template for addition into ENVO, as well as mapping to SWEET. These contributions we conducted following the ENVO Robot template and merge workflow created during the course of this and subsequent work. This issue is to document this work and create a new pull request to merge the contributions into ENVO.

@kaiiam kaiiam self-assigned this May 12, 2021
@kaiiam
Copy link
Contributor Author

kaiiam commented May 12, 2021

Action items to fix for this pull request in addition to the template which are expressed in Editor notes:

Ontology ID editors note
ENVO:01000818 obsolete this term. Replace by new term for planetary cryosphere.
ENVO:03000143 Make an axiom with composed primarily of frozen environmental materials.
ENVO:03000144 Replacement for ENVO:01000818
ENVO:00000484 Remove old definition: "Any enclosed water area in pack ice other than a lead ..." Remove old superclass lentic water body
ENVO:03000141 1) Add an axioms about A cryospheric layer which is primarily composed of snow. 2) Make a thickness of snow class using the Entity attribute DOSDP. thickness and inheres in some cryospheric layer. That class refers to GCW snow cover definition 3: The depth of snow on the ground, usually expressed in inches or centimeters. Link depth of snow to http://sweetontology.net/propSpaceThickness/SnowCover 3) We might want to make 3 class for ice cover which replicate the patterns used here for the GCW snow cover. Classes such as 1) Area of ice cover 2) ice layer 3) Depth of ice.
ENVO:03000116 Given that SWEET considers the term to be a thickness and ENVO currently considers it a mass of snow; I think we have a fundamental mismatch here. See ESIPFed/sweet#236 (comment).
ENVO:09200020 Make sure to make the DOSDP target for this term: https://github.com/EnvironmentOntology/envo/blob/master/src/envo/modules/entity_attribute.csv#L18
ENVO:01000896 This term exists, we're revising the definition with ESIP semantic Harmonization group, delete the old one. Consider adding axioms like PATO:"decreased temperature" to this and other cold forms of precipitation, such that we can create a grouping class to be filled by inference (with hail, sleet, snow, freezing rain). Something like precipitation below the liquid-solid phase transition point of water.
ENVO:3400007 Add axioms to nest the lake-effect snowfall process in this one
ENVO:00000429 Remove old definition "A thin, almost knife-like, ridge of rock which ..."

@kaiiam
Copy link
Contributor Author

kaiiam commented May 12, 2021

Doing this locally in a new PR which is called issue-1160, will push it once we resolve some of the above issues.

@kaiiam
Copy link
Contributor Author

kaiiam commented Jun 9, 2021

I've addressed all but the following of the above items, the remaining issues need to be addressed:

ENVO:03000143
ENVO:03000141
ENVO:03000116 (just consider maybe no action item)
ENVO:01000896 (maybe add axioms) 
ENVO:3400007 (unclear what needs to be done)

@kaiiam
Copy link
Contributor Author

kaiiam commented Jun 9, 2021

Only ENVO:03000141 remaining now. Last steps now to add classes for 1) Area of ice cover 2) ice layer 3) Depth of ice.

kaiiam added a commit that referenced this issue Jun 9, 2021
@kaiiam
Copy link
Contributor Author

kaiiam commented Jun 9, 2021

I see there are some more build errors a duplicated def and some others I'll fix this later.

@kaiiam
Copy link
Contributor Author

kaiiam commented Jun 10, 2021

Build is passing and ready for review see #1172.

pbuttigieg added a commit that referenced this issue Mar 16, 2022
Reviewed in ESIP Semantic Harmonization call with @rduerr @kaiiam and @pbuttigieg (PR reviewers) present as well as @wiegandn and @Garybc


* Run robot merge to add template terms to edit file address #1160

* Remove old def for ENVO:00000484

* Remove old superclass axiom for ENVO:00000484

* Remove old def for ENVO:01000896

* Remove old def for ENVO:00000429

* Depricate ENVO:01000818 and replace by ENVO:03000144

* Remove process superclass to fix reasoning error and add editor note to add process to ENVO:3400010 address #1160

* Remove superclass and APs for ENVO_01000818

* Add axiom to ENVO_03000143 address #1160

* Add to editor note on ENVO_03000143

* Fix ENVO_03000116

* Modify axioms on ENVO_01000896

* Add axiom for ENVO_3400007

* Add thickness of snow

* Fix axiom on ice mass to cryosphere

* Remove duplicate def for ENVO_01000880

* Remove duplicated def for ENVO_01000882

* Update editor note for ENVO_03000143

* Remove editor note on ENVO_00000484

* remove editor note for ENVO_09200020

* Fix freezing rain

* Fix hail

* Various fixes

* Add terms ENVO:3400014 ENVO:3400015 ENVO:3400016 ENVO:3400017 ENVO:3400018

* Remove def for freezing rain

* Fix defs

* Add dbxref aps to defs

* Fix water ice layer

* Remove def for snow layer

* Fix obsolete cryosphere

* Readd def for ENVO_03000141

* Fix axioms linking to old cryosphere

* Fix IDs in Ruth's Range by padding with 0's

* Fix extra <'s from merge conflict

* Fix final line from merge conflict

* Fix CURIEs for mass density of snow and depth of snow

* Run make entity_attribute.owl module

* Add subset tags for ENVO_09200022

* Add annotation properties to ENVO:09200023

* Remove old editor note from ENVO_00000429

* remove duplicated area of polar tundra with incorrect id ENVO_3400002

* remove duplicated area of alpine tundra with incorrect id ENVO_3400001

* Add axiom to planetary crosphere

* Fix polyna and sea ice layer

* obselete glacial feature

* Add axiom for hail

* Fix def and add axiom for hail stone

* update falling snow def and axioms

* update hailfall, fix axiom on hydrological process to extend beyond liquid phase

* add equivalence axiom for heavy snowfall

* reviewed ice pellet and added qualities

* reviewed icestorm and added axioms, revised glaze ice layer en passant

* revised and axiomatised lake-effect snowfall

* revised def for lake-effect snowstorm, avoiding superclass ambiguity

* revised defs and axioms for lambing storm and falling sleet

* revised and axiomatised arete

* fixed equivalence axiom on compound astronomical body part

* fixing reasoner errors around fossil fuel and biological pigment

Co-authored-by: Bill Duncan <[email protected]>
Co-authored-by: pbuttigieg <[email protected]>
@kaiiam
Copy link
Contributor Author

kaiiam commented Aug 6, 2022

This has been dealt with PR submitted and merged into the main branch. I think we can close this issue. It has the SWEET label to refer to our ESIP Semantic harmonization work.

@kaiiam kaiiam closed this as completed Aug 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant