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

Editing "marine bed", "shoreline", "sea surface layer" - addressing #1122 #1125

Merged
merged 8 commits into from
May 12, 2021

Conversation

meichen-liu
Copy link
Contributor

Dear editors,

I have edited terms "marine bed", "shoreline", and "sea surface floor" as mentioned in issue #1122. I also added German, Italian, Spanish and Mandarin interpretations to them, and include them in envoPlastics subset.

As mentioned in the issue, we are considering to obsolete sea floor and ocean floor as they are synonyms for marine bed. But we are not sure what goes into making that decision and also how to do it.

Another point I notice while editing shoreline is its database_cross_reference is a stand alone annotation. As they are not all website line but text (e.g. FTT:503, FTT240), I wasn't sure is it appropriate to change them under definition.

Thank you!

@raissameyer
Copy link
Contributor

Another point I notice while editing shoreline is its database_cross_reference is a stand alone annotation. As they are not all website line but text (e.g. FTT:503, FTT240), I wasn't sure is it appropriate to change them under definition.

Good point - we had the same thing come up over here. Adding those stand alone annotations as annotations on the definitions would be much appreciated!

@raissameyer
Copy link
Contributor

Good point - we had the same thing come up over here. Adding those stand alone annotations as annotations on the definitions would be much appreciated!

I stand corrected! @cmungall just explained here that these class annotations are mappings, rather than definition sources and should thus remain as standalone annotations on the class level.

Copy link
Member

@pbuttigieg pbuttigieg left a comment

Choose a reason for hiding this comment

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

Definition revision and placement for shoreline

src/envo/envo-edit.owl Show resolved Hide resolved
@meichen-liu
Copy link
Contributor Author

Hi @pbuttigieg

I have edited the definition of shoreline accordingly (see above). Thanks for your semantics help. Do you have any other suggestions or comments?

@meichen-liu meichen-liu requested a review from pbuttigieg May 11, 2021 13:43
Copy link
Member

@pbuttigieg pbuttigieg left a comment

Choose a reason for hiding this comment

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

First round of changes requested

@@ -11198,12 +11198,21 @@ SubClassOf(<http://purl.obolibrary.org/obo/ENVO_00000481> ObjectSomeValuesFrom(<

# Class: <http://purl.obolibrary.org/obo/ENVO_00000482> (sea floor)

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_00000482> "The ground surface that lies beneath a sea."^^xsd:string)
AnnotationAssertion(Annotation(<http://www.geneontology.org/formats/oboInOwl#hasDbXref> <https://en.wikipedia.org/wiki/Seabed>) 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_00000482> "A marine bed which is covered by marine sediments."^^xsd:string)
Copy link
Member

Choose a reason for hiding this comment

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

A marine bed which is covered by marine sediments.

Not sure about this. Isn't the bed boundary defined by the sediments? I suppose the confusion comes from the use of the ambiguous word "ground" in submerged bed.

And there is a difference between oceans and seas which supports the use of both sea floor and ocean floor.

src/envo/envo-edit.owl Outdated Show resolved Hide resolved
@@ -33524,9 +33545,14 @@ SubClassOf(<http://purl.obolibrary.org/obo/ENVO_01001570> ObjectSomeValuesFrom(<
# Class: <http://purl.obolibrary.org/obo/ENVO_01001581> (sea surface layer)

AnnotationAssertion(<http://purl.obolibrary.org/obo/IAO_0000115> <http://purl.obolibrary.org/obo/ENVO_01001581> "A surface layer of a water body which is part of an ocean or sea.")
Copy link
Member

Choose a reason for hiding this comment

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

A surface layer of a water body which is part of an ocean or sea.

I'd say: "A surface layer which is part of an ocean or sea."

This is a semantic ambiguity fix, as SSL is used to refer to the surfaces of oceans too. That should be added as a comment.

In fact, even with seafloor and ocean floor, we should add related synonyms for the other term, with annotation properties on those synonyms (comments) saying they're there as these terms are used so loosely.

@meichen-liu
Copy link
Contributor Author

Hi @pbuttigieg,

Thanks for clarify the difference between sea and ocean!

I have edited the definition accordingly. Please let me know if you have further suggestions.

@meichen-liu meichen-liu requested a review from pbuttigieg May 12, 2021 13:48
src/envo/envo-edit.owl Outdated Show resolved Hide resolved
src/envo/envo-edit.owl Show resolved Hide resolved
src/envo/envo-edit.owl Outdated Show resolved Hide resolved
Copy link
Member

@pbuttigieg pbuttigieg left a comment

Choose a reason for hiding this comment

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

Thanks @meichen-liu! looks good!

Copy link
Contributor

@raissameyer raissameyer left a comment

Choose a reason for hiding this comment

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

Thanks @meichen-liu. Will merge : )

@raissameyer raissameyer merged commit 413fdb7 into master May 12, 2021
@raissameyer raissameyer deleted the issue-1122 branch May 12, 2021 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants