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

Adding missing defs detected in #1159 #1163

Merged
merged 11 commits into from Mar 7, 2022
Merged

Adding missing defs detected in #1159 #1163

merged 11 commits into from Mar 7, 2022

Conversation

pbuttigieg
Copy link
Member

Description and progress notes in #1159

@pbuttigieg pbuttigieg added the bug label May 14, 2021
@pbuttigieg pbuttigieg self-assigned this May 14, 2021
@pbuttigieg pbuttigieg linked an issue May 14, 2021 that may be closed by this pull request
@pbuttigieg pbuttigieg added this to In Progress in betterTech May 14, 2021
@pbuttigieg pbuttigieg marked this pull request as draft May 14, 2021 11:49
@pbuttigieg pbuttigieg marked this pull request as ready for review May 15, 2021 10:55
@pbuttigieg
Copy link
Member Author

Setting status as "ready for review" to preview merge conflict status

@pbuttigieg
Copy link
Member Author

pbuttigieg commented May 18, 2021

Could you tell me how to get a graphical view of the proposed hierarchy?
I would like to get this reviewed by my local soil/landscape subject-matter-experts.

@dr-shorthair
I sketched one here #1159 (comment) using VUE, but you could also use a Protégé plugin like OntoGraph to generate a less neat overview (reflecting the work in progress state of things)

image

@pbuttigieg
Copy link
Member Author

pbuttigieg commented May 18, 2021

Prep for some merge conflicts when the new release hits the master branch....

Edit: refreshingly, no merge conflicts reported

@pbuttigieg
Copy link
Member Author

pbuttigieg commented Jun 11, 2021

A new non-content error emerges...
image

But one build passed...
image

@pbuttigieg
Copy link
Member Author

There are some stragglers (marine reef bits) that need more thought and may merge in with the CMECS terms

@kaiiam kaiiam mentioned this pull request Feb 10, 2022
cmungall added a commit that referenced this pull request Feb 10, 2022
This should fix the build issues with:

 * #1172. 
 * #1163
@wdduncan
Copy link
Member

@pbuttigieg @kaiiam @cmungall The PR now passes the travis_test. Are you ready to merge?

@pbuttigieg pbuttigieg merged commit 504ada3 into master Mar 7, 2022
@@ -5448,7 +5464,7 @@ SubClassOf(<http://purl.obolibrary.org/obo/ENVO_00000054> <http://purl.obolibrar

# Class: <http://purl.obolibrary.org/obo/ENVO_00000055> (saline evaporation pond)

AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "https://en.wikipedia.org/wiki/Salt_evaporation_ponds"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000055> "A shallow man-made pond designed to produce salt from sea water. The seawater is fed into large ponds and water is drawn out through natural evaporation which allows the salt to be subsequently harvested."^^xsd:string)
AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "https://en.wikipedia.org/wiki/Salt_evaporation_ponds"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000055> "A pond which is 1) shallow, 2) man-made, and 3) designed to produce salt from seawater through evaporation."@en)
Copy link

@smrgeoinfo smrgeoinfo Mar 7, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there are also saline evaporation ponds adjacent to salt lakes in continental areas, e.g. Great Salt Lake, Utah, or to evaporate saline water used to dissolve subsurface salt deposits e.g. around the Luke salt in the Phoenix area, Arizona.

@@ -5390,19 +5406,19 @@ SubClassOf(<http://purl.obolibrary.org/obo/ENVO_00000050> <http://purl.obolibrar

# Class: <http://purl.obolibrary.org/obo/ENVO_00000051> (hot spring)

AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "https://en.wikipedia.org/wiki/Hot_spring"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000051> "A spring that is produced by the emergence of geothermally-heated groundwater from the Earth's crust."^^xsd:string)
AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "https://en.wikipedia.org/wiki/Hot_spring"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000051> "A spring through which groundwater, heated by geothermal energy, flows."@en)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why complicate with 'groundwater' and 'heated by geothermal energy'. simplify to
'a spring that emits water that is warmer than the ambient ground temperature'.
Is 'warm spring' a concept in ENVO. Generally 'hot' connotes a temperature above something like human body temperature.

@@ -5751,7 +5766,7 @@ AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#has
AnnotationAssertion(<http://purl.obolibrary.org/obo/IAO_0000116> <http://purl.obolibrary.org/obo/ENVO_00000070> "\"constructed\" should probably be made something like a quality and this class obsoleted or filled only by inference")
AnnotationAssertion(<http://www.geneontology.org/formats/oboInOwl#hasExactSynonym> <http://purl.obolibrary.org/obo/ENVO_00000070> "constructed feature")
AnnotationAssertion(rdfs:label <http://purl.obolibrary.org/obo/ENVO_00000070> "human construction"^^xsd:string)
EquivalentClasses(<http://purl.obolibrary.org/obo/ENVO_00000070> ObjectIntersectionOf(<http://purl.obolibrary.org/obo/ENVO_01001813> ObjectSomeValuesFrom(<http://purl.obolibrary.org/obo/RO_0002353> <http://purl.obolibrary.org/obo/ENVO_01000996>)))
SubClassOf(<http://purl.obolibrary.org/obo/ENVO_00000070> <http://purl.obolibrary.org/obo/ENVO_01001813>)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah, the joy of opaque identifiers. Can't evaluate this statement without resolving the URI...

@@ -5926,7 +5941,7 @@ SubClassOf(<http://purl.obolibrary.org/obo/ENVO_00000079> <http://purl.obolibrar

# Class: <http://purl.obolibrary.org/obo/ENVO_00000080> (mountain range)

AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "USGS:SDTS"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000080> "A series of connected and aligned mountains or mountain ridges."^^xsd:string)
AnnotationAssertion(<http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000080> "A range of mounts which is dominated by mountains."^^xsd:string)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

circular. a mountain range is a range.

@@ -6119,12 +6128,11 @@ SubClassOf(<http://purl.obolibrary.org/obo/ENVO_00000088> ObjectSomeValuesFrom(<

# Class: <http://purl.obolibrary.org/obo/ENVO_00000089> (inland cliff)

AnnotationAssertion(<http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000089> "A cliff which is not adjacent to a major water body."@en)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gramatically should use 'that' instead of 'which'
a cliff that is not adjacent....

SubClassOf(<http://purl.obolibrary.org/obo/ENVO_00000091> ObjectSomeValuesFrom(<http://purl.obolibrary.org/obo/BFO_0000050> <http://purl.obolibrary.org/obo/ENVO_00000304>))
SubClassOf(<http://purl.obolibrary.org/obo/ENVO_00000091> ObjectSomeValuesFrom(<http://purl.obolibrary.org/obo/RO_0002473> ObjectIntersectionOf(<http://purl.obolibrary.org/obo/ENVO_00002007> ObjectSomeValuesFrom(<http://purl.obolibrary.org/obo/RO_0002473> ObjectUnionOf(<http://purl.obolibrary.org/obo/ENVO_00002139> <http://purl.obolibrary.org/obo/ENVO_00002140> <http://purl.obolibrary.org/obo/ENVO_01000017> <http://purl.obolibrary.org/obo/ENVO_01000018> <http://purl.obolibrary.org/obo/UBERON_0006612>)))))

# Class: <http://purl.obolibrary.org/obo/ENVO_00000092> (sea beach)

AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "https://en.wikipedia.org/wiki/Beach"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000092> "A landform consisting of loose rock particles such as sand, gravel, shingle, pebbles, cobble, or even shell fragments along the shoreline of a sea."^^xsd:string)
AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "https://en.wikipedia.org/wiki/Beach"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000092> "A beach which is part of a sea shore."^^xsd:string)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the original definition is much better. To understand this revised definition, I have to figure out what you mean by 'beach' and what you mean by 'sea shore'....

AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "MA:ma"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000093> "A landform consisting of loose rock particles such as sand, gravel, shingle, pebbles, cobble, or even shell fragments along the shoreline of a lake."^^xsd:string)
AnnotationAssertion(<http://www.geneontology.org/formats/oboInOwl#hasOBONamespace> <http://purl.obolibrary.org/obo/ENVO_00000093> "ENVO"^^xsd:string)
AnnotationAssertion(<http://www.geneontology.org/formats/oboInOwl#id> <http://purl.obolibrary.org/obo/ENVO_00000093> "ENVO:00000093"^^xsd:string)
AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "MA:ma"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000093> "A beach which is part of a lake shore."^^xsd:string)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use 'that'

@@ -6392,6 +6396,7 @@ SubClassOf(<http://purl.obolibrary.org/obo/ENVO_00000103> ObjectSomeValuesFrom(<

# Class: <http://purl.obolibrary.org/obo/ENVO_00000104> (undersea feature)

AnnotationAssertion(<http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000104> "A hydrographic feature which is located below the surface of a marine water body."@en)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmmm, wouldn't features on the sea floor (sand waves, biogenic mounds, rift valleys, seamounts, reefs) also be considered undersea features? these are not hydrographic.

@@ -6819,13 +6822,13 @@ SubClassOf(<http://purl.obolibrary.org/obo/ENVO_00000120> <http://purl.obolibrar

# Class: <http://purl.obolibrary.org/obo/ENVO_00000121> (artificial channel)

AnnotationAssertion(<http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000121> "A channel of a watercource which has been constructed by humans or their technologies."@en)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think 'a channel constructed by humans' is all you need.

@@ -6865,29 +6868,25 @@ SubClassOf(<http://purl.obolibrary.org/obo/ENVO_00000124> <http://purl.obolibrar

# Class: <http://purl.obolibrary.org/obo/ENVO_00000125> (mineral spring)

AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "https://en.wikipedia.org/wiki/Mineral_spring"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000125> "A spring whose water contains a significant amount of dissolved minerals, that derive from the rocks through which the water flows."^^xsd:string)
AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "https://en.wikipedia.org/wiki/Mineral_spring"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000125> "A spring which water contains a significant amount of dissolved minerals, that derive from the rocks through which the water flows."^^xsd:string)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

don't complicate with stuff that should be in notes.
'A spring that emits water containing a significant amount of dissolved minerals'.
Of course a purist might want to know what you mean by 'minerals'

SubClassOf(<http://purl.obolibrary.org/obo/ENVO_00000127> ObjectSomeValuesFrom(<http://purl.obolibrary.org/obo/RO_0000086> <http://purl.obolibrary.org/obo/ENVO_01000230>))

# Class: <http://purl.obolibrary.org/obo/ENVO_00000128> (dry valley)

AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "https://en.wikipedia.org/wiki/Dry_valley"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000128> "A valley that no longer has a surface flow of water. Typically found in either Karst (limestone) or chalk terrain."^^xsd:string)
AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "https://en.wikipedia.org/wiki/Dry_valley"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000128> "A valley which no longer has water flowing or accumulating over its surface."^^xsd:string)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why 'no longer'???
A valley that does not contain perennial surface water.

AnnotationAssertion(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> <http://purl.obolibrary.org/obo/ENVO_00000128> "https://en.wikipedia.org/wiki/Dry_valley"^^xsd:string)
AnnotationAssertion(<http://www.geneontology.org/formats/oboInOwl#hasOBONamespace> <http://purl.obolibrary.org/obo/ENVO_00000128> "ENVO"^^xsd:string)
AnnotationAssertion(<http://www.geneontology.org/formats/oboInOwl#id> <http://purl.obolibrary.org/obo/ENVO_00000128> "ENVO:00000128"^^xsd:string)
AnnotationAssertion(rdfs:comment <http://purl.obolibrary.org/obo/ENVO_00000128> "Typically found in either Karst (limestone) or chalk terrain."@en)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what about dry valleys in deserts?


# Class: <http://purl.obolibrary.org/obo/ENVO_00000150> (coral reef)

AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "https://en.wikipedia.org/wiki/Coral_reef"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000150> "Aragonite structures produced by living organisms, found in shallow, marine waters with little nutrients in the water."^^xsd:string)
AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "https://en.wikipedia.org/wiki/Coral_reef"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000150> "A marine reef which is composed primarily of aragonite structures produced by living organisms, found in shallow, low-nutrient waters."@en)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd suggest 'calcium-rich carbonate minerals'. If its a 'coral' reef, don't the 'living organisms' have to be members of that genus? Is there such a thing as a 'non-marine' reef??


# Class: <http://purl.obolibrary.org/obo/ENVO_00000175> (karst)

AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "https://en.wikipedia.org/wiki/Karst"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000175> "A feature shaped by the dissolution of a soluble layer or layers of bedrock, usually carbonate rock such as limestone or dolomite. These landscapes display distinctive surface features and underground drainages, and in some examples there may be little or no surface drainage."^^xsd:string)
AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "https://en.wikipedia.org/wiki/Karst"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000175> "A surface landform which is formed by the dissolution of a soluble layer or layers of bedrock."^^xsd:string)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

simplify
' ... dissolution of soluble bedrock'

@@ -9746,14 +9735,13 @@ SubClassOf(<http://purl.obolibrary.org/obo/ENVO_00000324> <http://purl.obolibrar

# Class: <http://purl.obolibrary.org/obo/ENVO_00000325> (polje)

AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "http://wasg.iinet.net.au/terminol.html"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000325> "A large closed depression draining underground, witha flat floor across which there may be an intermittent or perennial stream and which may be liable to flood and become a lake. The floor makes a sharp break with parts of surrounding slopes."^^xsd:string)
AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "http://wasg.iinet.net.au/terminol.html"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000325> "A large closed depression draining underground, with a flat floor across which there may be an intermittent or perennial stream and which may be liable to flood and become a lake. The floor makes a sharp break with parts of surrounding slopes."^^xsd:string)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a note indicating the connotation of karst association would be useful.


AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "https://en.wikipedia.org/wiki/Harbor"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000463> "A place where ships may shelter from the weather or are stored."^^xsd:string)
AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "https://en.wikipedia.org/wiki/Harbor"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000463> "A landform which provides shelter from atmospheric weather and marine currents, waves, and other dynamics."^^xsd:string)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perhaps 'a shoreline landform...' otherwise any cave would meet this definition.

SubClassOf(<http://purl.obolibrary.org/obo/ENVO_00000508> ObjectSomeValuesFrom(<http://purl.obolibrary.org/obo/BFO_0000051> <http://purl.obolibrary.org/obo/ENVO_00002000>))

# Class: <http://purl.obolibrary.org/obo/ENVO_00000509> (marine terrace)

AnnotationAssertion(<http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000509> "A terrace which is located in a marine water body."@en)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think marine terraces are located along the shore line of a marine water body; many of them are subaerial, not submerged.

@@ -12617,53 +12592,51 @@ AnnotationAssertion(owl:deprecated <http://purl.obolibrary.org/obo/ENVO_00000886

# Class: <http://purl.obolibrary.org/obo/ENVO_00000887> (large river biome)

AnnotationAssertion(<http://www.geneontology.org/formats/oboInOwl#hasOBONamespace> <http://purl.obolibrary.org/obo/ENVO_00000887> "ENVO"^^xsd:string)
AnnotationAssertion(<http://www.geneontology.org/formats/oboInOwl#id> <http://purl.obolibrary.org/obo/ENVO_00000887> "ENVO:00000887"^^xsd:string)
AnnotationAssertion(<http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000887> "A freshwater river biome which is determined by a large river."@en)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i assume 'large river' is defined in ENVO? what does 'determined by a river' mean?

AnnotationAssertion(<http://www.geneontology.org/formats/oboInOwl#inSubset> <http://purl.obolibrary.org/obo/ENVO_00000887> "wwfBiome")
AnnotationAssertion(rdfs:comment <http://purl.obolibrary.org/obo/ENVO_00000887> "\"Large\" is ambiguous. For details on \"Large rivers\" (e.g. the Mekong river) see http://worldwildlife.org/biomes/large-river-ecosystems This class will be replaced with a less ambiguous class."^^xsd:string)
AnnotationAssertion(rdfs:label <http://purl.obolibrary.org/obo/ENVO_00000887> "large river biome"^^xsd:string)
SubClassOf(<http://purl.obolibrary.org/obo/ENVO_00000887> <http://purl.obolibrary.org/obo/ENVO_01000253>)

# Class: <http://purl.obolibrary.org/obo/ENVO_00000888> (large river headwater biome)

AnnotationAssertion(<http://www.geneontology.org/formats/oboInOwl#hasOBONamespace> <http://purl.obolibrary.org/obo/ENVO_00000888> "ENVO"^^xsd:string)
AnnotationAssertion(<http://www.geneontology.org/formats/oboInOwl#id> <http://purl.obolibrary.org/obo/ENVO_00000888> "ENVO:00000888"^^xsd:string)
AnnotationAssertion(<http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000888> "A freshwater river biome which is determined by the headwaters of a large river."@en)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

since the headwaters of a large river a likely to be small rivers, how is this different from a 'small river biome'?


# Class: <http://purl.obolibrary.org/obo/ENVO_00000889> (large river delta biome)

AnnotationAssertion(<http://www.geneontology.org/formats/oboInOwl#hasOBONamespace> <http://purl.obolibrary.org/obo/ENVO_00000889> "ENVO"^^xsd:string)
AnnotationAssertion(<http://www.geneontology.org/formats/oboInOwl#id> <http://purl.obolibrary.org/obo/ENVO_00000889> "ENVO:00000889"^^xsd:string)
AnnotationAssertion(<http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00000889> "A freshwater river biome which is determined by the delta of a large river."@en)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seem like language like 'a biome hosted by the delta of a river' would be better way to define. Isn't a biome hosted by some environment?

@@ -12800,16 +12771,16 @@ SubClassOf(<http://purl.obolibrary.org/obo/ENVO_00001999> ObjectSomeValuesFrom(<

# Class: <http://purl.obolibrary.org/obo/ENVO_00002000> (slope)

AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "MA:ma"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00002000> "A solid astronomical body part which is part of the planetary surface between the peak of an elevation or the bottom of a depression and relatively flat surrounding land."^^xsd:string)
AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "MA:ma"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00002000> "A solid astronomical body part which is part of the planetary surface between the peak of an elevation or the bottom of a depression and relatively flat surrounding land."^^xsd:string)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't the essential feature of a slope the fact that is has a surface elevation gradient greater than something like 5 degrees? This definition would include flat terraces between a peak and a valley as slopes.

@@ -13519,26 +13481,27 @@ SubClassOf(<http://purl.obolibrary.org/obo/ENVO_00002118> <http://purl.obolibrar

# Class: <http://purl.obolibrary.org/obo/ENVO_00002119> (alkaline hot spring)

AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "MA:ma"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00002119> "A hot spring whose water has an alkaline pH."^^xsd:string)
AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> <https://orcid.org/0000-0002-4366-3088>) Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> "MA:ma"^^xsd:string) <http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_00002119> "A alkaline spring through which groundwater, heated by geothermal energy, flows."@en)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

original definition is better.

@kaiiam
Copy link
Contributor

kaiiam commented Mar 9, 2022

Still some missing definitions e.g. glaze ENVO_01001849. I'll try to use robot export to see what classes are missing definitions at some point.

@pbuttigieg pbuttigieg deleted the issue-1159 branch August 15, 2022 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
betterTech
In Progress
Development

Successfully merging this pull request may close these issues.

Terms missing definitions
6 participants