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

Create synonym pipeline #107

Merged
merged 3 commits into from
Jul 15, 2022
Merged

Create synonym pipeline #107

merged 3 commits into from
Jul 15, 2022

Conversation

matentzn
Copy link
Contributor

@megbalk This is how it would look like

@rays22 simply review the synonyms.tsv file that @megbalk provided. You can merge this PR if you think her synonyms are good choices!

@matentzn matentzn requested a review from rays22 June 28, 2022 17:47
@@ -0,0 +1,5 @@
Ontology ID Label Exact synonym Close Synonym Related synonym Broad synonym Narrow synonym Contributor
ID A oboInOwl:hasExactSynonym SPLIT=| A oboInOwl:hasCloseSynonym SPLIT=| A oboInOwl:hasRelatedSynonym SPLIT=| A oboInOwl:hasBroadSynonym SPLIT=| A oboInOwl:hasNarrowSynonym SPLIT=| AI dc:contributor SPLIT=|
http://purl.obolibrary.org/obo/OBA_VT0010023 forelimb length forearm length ulna length https://orcid.org/0000-0003-2699-3066
Copy link
Contributor

Choose a reason for hiding this comment

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

'ulna length' does not look right as a narrow synonym.

ID A oboInOwl:hasExactSynonym SPLIT=| A oboInOwl:hasCloseSynonym SPLIT=| A oboInOwl:hasRelatedSynonym SPLIT=| A oboInOwl:hasBroadSynonym SPLIT=| A oboInOwl:hasNarrowSynonym SPLIT=| AI dc:contributor SPLIT=|
http://purl.obolibrary.org/obo/OBA_VT0010023 forelimb length forearm length ulna length https://orcid.org/0000-0003-2699-3066
http://purl.obolibrary.org/obo/OBA_VT1000062 chest circumference chest girth|girth of chest https://orcid.org/0000-0003-2699-3066
http://purl.obolibrary.org/obo/OBA_VT1000425 skin mass body mass https://orcid.org/0000-0003-2699-3066
Copy link
Contributor

Choose a reason for hiding this comment

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

'body mass' seems unnecessary as a synonym here.

@rays22 rays22 merged commit 692470c into master Jul 15, 2022
@rays22 rays22 deleted the synonym-pipeline branch July 15, 2022 15:22
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.

3 participants