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

ENVO cryosphere wrongly set as a subclass of glacial feature, etc. #1291

Closed
wiegandn opened this issue Feb 9, 2022 · 12 comments
Closed

ENVO cryosphere wrongly set as a subclass of glacial feature, etc. #1291

wiegandn opened this issue Feb 9, 2022 · 12 comments
Assignees
Labels

Comments

@wiegandn
Copy link

wiegandn commented Feb 9, 2022

@rduerr @pbuttigieg @kaiiam

  1. When searching ENVO for cryosphere (and also seen searching for ice calf (and perhaps elsewhere too)), cryosphere is listed as a subclass of glacial feature, which is incorrect.
  2. Under ice calf, fixing the above may also fix ice mass being both a subclass and sibling of cryosphere, which also would need to be fixed.
  3. Further, under ice calf, ice mass should probably be a sibling of glacial feature and not a subclass. (maybe a separate issue)
@cmungall
Copy link
Member

Hi @wiegandn - I agree this is a problem

I tracked this down to an incorrect logical definition. Here is the reasoner explanation (if this doesn't mean anything to you don't worry, the editors will take care of this):

image

This is due to an overly broad logical definition of glacial feature, as being equivalent to:

material entity and composed primarily of some (snow or water ice)

Note that under this definition all of the following things are glacial features:

  • a popsicle
  • a cryosphere
  • a snowball
  • a snowman
  • an ice mass

@kaiiam
Copy link
Contributor

kaiiam commented Feb 21, 2022

Thank you very much @wiegandn for the comments and @cmungall for checking this.

The version currently published doesn't have all the changes we made/want to make, they are in my Pull Request (PR) #1172. I'll address the issues noted above one by one with a reasoned version of the latest Pull Request.

  1. cryosphere is listed as a subclass of glacial feature, which is incorrect.

In the PR we obsoleted the old cryosphere and replaced it with new terms for cryosphere and planetary cryosphere (with a different equivalence axiom). Such changes (for now) fixed that problem of cryosphere being subclass to glacial feature

image

  1. ice mass being both a subclass and sibling of cryosphere

This is fixed as well.

image

  1. ice mass should probably be a sibling of glacial feature and not a subclass. (maybe a separate issue)

Given the current definitions the way we have it now (ice mass subclass to glacial feature) makes sense

where glacial feature is

A hydrographic feature characterized by the dominance of snow or ice.

and ice mass is

A mass of water ice.

@wiegandn if you have any suggestions for better definitions/hierarchical placement of those terms please let us know.

[The] overly broad logical definition of glacial feature

Yes @cmungall I think it might be good to review these and perhaps also other EQ axioms in ENVO.

In this pull request the relevant EQ axioms are on

cryosphere

'composed primarily of' some 
    ('environmental material'
     and ('has quality' some frozen))

and glacial feature

'material entity'
 and ('composed primarily of' some 
    ('water ice' or snow))

@cmungall let us know if you think the axiom on cryosphere is also more broad then intended. Thanks for illustrating the issue with the axiom on glacial feature.

@smrgeoinfo
Copy link

the EQ axioms are tricky. Are the ice cubes in my freezer composed of some 'environmental material'? I don't think they'd be considered part of the cryosphere. Seems like you need something like 'part of the Earth composed primarily of water ice or snow formed by natural processes'

@smrgeoinfo
Copy link

Are moraines, outwash plains, eskers glacial features? They are not composed of ice or snow. The Oxford Languages definition of 'glacial' is useful start "relating to, resulting from, or denoting the presence or agency of ice, especially in the form of glaciers." I don't think the term 'glacier' should be used in a definition of glacial-- its circular. Maybe something like 'presence or agency of thick accumulations of naturally occurring water ice'

@wiegandn
Copy link
Author

wiegandn commented Feb 21, 2022 via email

@rduerr
Copy link
Contributor

rduerr commented Feb 22, 2022

@wiegandn I agree that there are a lot of issues with the cryosphere structures in ENVO at this moment and even after this next release of ENVO. The question is should these all be fixed at once or is fixing a few at a time sufficient. My take is that fixes are going to have to be piecemeal since no one is funded to make a global set of fixes and even if they were, that might take years! Especially with so many cooks in the kitchen and no uniform understanding of ontological best practices (especially with how to deal with multiple definitions for a term and how to resolve that by making several terms that make the distinctions plain and computable).

I think a new issue should be written about the definition of "glacial feature", since there are clearly at least two different types of glacial features:

  1. Features of a glacier (toe, ablation zone, etc.)
  2. Landforms created by glaciers (eskers, moraines, etc.) which may no longer have any ice at all (i.e., the glaciers are no longer around)!

The current definition is clearly referring to the first of these and clearly predates establishing annotation conventions for ENVO, since it isn't even dated nor does it have any sort of author or creator. Even so the definition could be improved since it clearly doesn't exclude sea ice or cloud related ice terms (e.g., frazil ice and cirrocumulus clouds are not a glacier features).

However, this issue is about cryosphere being a subclass of glacial feature. Since that is fixed (at least in the #1172 pull request); I say that once that pull request is approved, then this issue should be closed. Specific new issues related to other problems should then be created. Yes, a lot of individual issues; but I don't believe that the whole tree can be addressed at once without real funding for someone!

@wiegandn
Copy link
Author

wiegandn commented Feb 22, 2022 via email

@rduerr
Copy link
Contributor

rduerr commented Feb 22, 2022

@wiegandn yup - lot's of new issues to write, though maybe (unless you can look at the #1072 version of the ontology) you should wait to write those until after #1072 is released.

@kaiiam kaiiam mentioned this issue Feb 22, 2022
@kaiiam
Copy link
Contributor

kaiiam commented Feb 22, 2022

I agree with @rduerr, hopefully many of @wiegandn's concerns should be fixed when #1172 is released, but there will of course be more issues.

I agree we might want to sort out glacial feature if you @wiegandn feel it's really wrong. I think it's just a simple grouping class for anything having to do with an existing glacier, however as @cmungall mentioned the EQ axiom could use review. Although the current axiom is helping to infer a lot of subclasses to glacial feature, as Chris mentioned something like a popsicle would also get placed there.

image

If anyone has a suggestion on what to change perhaps the axiom @cmungall we can add it to #1172 otherwise I think we should review and merge that PR for now.

@cmungall
Copy link
Member

cmungall commented Feb 22, 2022 via email

@kaiiam
Copy link
Contributor

kaiiam commented Mar 9, 2022

Thanks @cmungall and @wiegandn for the pointing out the issues with the axiom on glacial feature, we've obsoleted it now in #1172.

@kaiiam kaiiam added the SWEET label Aug 6, 2022
@kaiiam
Copy link
Contributor

kaiiam commented Aug 6, 2022

Although there were some interesting points raised here I believe we have dealt with them and have fixed the core issues in a PR. I think this can now be closed. I've added the SWEET label to keep this issue reference-able with our ESIP Semantic Harmonization SWEET-ENVO cryosphere mapping 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
Labels
Projects
None yet
Development

No branches or pull requests

5 participants