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

Issue 1409 pull request #1412

Merged
merged 16 commits into from Aug 26, 2023
Merged

Issue 1409 pull request #1412

merged 16 commits into from Aug 26, 2023

Conversation

rduerr
Copy link
Contributor

@rduerr rduerr commented Mar 25, 2023

Please check this over carefully as it was cut/paste edited!!! Also, note the 2 undone items which need someone else to complete (or instructions to me)...

@rduerr rduerr requested review from cmungall and a team March 25, 2023 01:43
Copy link
Collaborator

@matentzn matentzn left a comment

Choose a reason for hiding this comment

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

These are some bigger syntactic issues, when all of these are fixed, I will run a validator as well.

mappings/sweet-envo_cryosphere.sssom.tsv Outdated Show resolved Hide resolved
mappings/sweet-envo_cryosphere.sssom.tsv Outdated Show resolved Hide resolved
mappings/sweet-envo_cryosphere.sssom.tsv Outdated Show resolved Hide resolved
mappings/sweet-envo_cryosphere.sssom.tsv Outdated Show resolved Hide resolved
mappings/sweet-envo_cryosphere.sssom.tsv Outdated Show resolved Hide resolved
mappings/sweet-envo_cryosphere.sssom.tsv Outdated Show resolved Hide resolved
rduerr and others added 9 commits May 10, 2023 13:35
Co-authored-by: Nico Matentzoglu <nicolas.matentzoglu@gmail.com>
Co-authored-by: Nico Matentzoglu <nicolas.matentzoglu@gmail.com>
Co-authored-by: Nico Matentzoglu <nicolas.matentzoglu@gmail.com>
Co-authored-by: Nico Matentzoglu <nicolas.matentzoglu@gmail.com>
Co-authored-by: Nico Matentzoglu <nicolas.matentzoglu@gmail.com>
Co-authored-by: Nico Matentzoglu <nicolas.matentzoglu@gmail.com>
* Create sssom-qc.yml

* Create Makefile
ENVO:01001509 terrestrial ice calf soreac:Calf calf skos:broadMatch semapv:ManualMappingCuration SWEET Calf is less specific than ENVO terrestrial ice calf
ENVO:01001611 thawing process soprocsc:Thawing thawing skos:closeMatch semapv:ManualMappingCuration Candidate Exact match if SWEET adopts the ENVO definition.
ENVO:03000085 thermokarst sorealg:Thermokarst thermokarst skos:closeMatch semapv:ManualMappingCuration In SWEET this is a landform, which is accurate - need to compare upper level layers to determine if an exact match could be made if SWEET adopted the ENVO definition
ENVO:00000254 till plain sorealg:TillPlain till plain skos:relatedMatch semapv:ManualMappingCuration Despite having identical definitions, the SWEET till plain is a geometrical object; while the ENVO till plain is a material entity
Copy link
Member

Choose a reason for hiding this comment

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

Hmm, I think this should be close? This is the sweet concept:

image

I have seen many cases where sweet confuses things under geometrical branches but this doesn't seem to be one of them?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

But LandRegion is a subclass of http://sweetontology.net/reprSpaceGeometry/GeometricalObject which is a subclass of http://sweetontology.net/reprMath/NumericalEntity! So now I am really confused... The ENVO version of a till plain is not a numerical entity!!!

Copy link
Member

Choose a reason for hiding this comment

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

@rduerr where are you seeing that http://cor.esipfed.org/ont?iri=http://sweetontology.net/realmLandform/LandRegion is a subclass of GeometricalObject? The OAK diagram above shows the complete paths to root, and this is consistent with the URIs.

@cmungall
Copy link
Member

This PR has been open since March. I didn't mean to block it with my comment. I think there are issues in the mapping methodology but it's better we merge and then iteratively improve.

@cmungall cmungall merged commit 204d190 into master Aug 26, 2023
2 checks passed
@cmungall cmungall deleted the issue-1409 branch August 26, 2023 01:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants