-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add high level classification #131
Conversation
Address #129
Great job @katiermullen
|
@sabrinatoro thank you for your review!
done! A couple of questions remain for us to address on a call.
Looking forward to it.
Will update and refresh.
Can I make the SC of the 'Ass breed' and 'Horse breed' be 'Equid breed' VBO:0400033? This way, the high level classification will be apparent in the inferred and asserted views of the ontology. Or is there another way I should address this for Equid, Bird, Poultry breed, etc?
can't wait! |
@matentzn could you please review this PR with emphasis on the high level classification and update to the ontology root term. @sabrinatoro has kindly offered to review the other changes that this PR encompasses. Thank you!!! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is too much to review here effectively but too notes:
- The logical equivalence axiom for "vertebrate breed" is circular and should be removed
- The human definition of "South American camelid breed" is "Vertebrate breeds including Llama, Guanaco, Vicuña and Alpaca breeds." That is not a good definition, at best a "description". A definition should always roughly follow the "genus+differentiae" model. The word "including" is a real risk for a definition as the enumeration is expected to be exhaustive.
What other specific questions do you have for me?
Improve definitions, add ECs, and new high level classifications for duck breed, pheasant breed and partridge breed.
@matentzn Thank you for your review comments and further explanations today! Much appreciated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is fantastic job, @katiermullen !!!
Thank you!
I will merge and ask Nico to update the monarch VBO OLS instance
Address #129
The inferred breed hierarchy is classifying correctly except for the South American camelid breeds - Llama breed, Alpaca breed, Vicuna breed and Guanaco breed should be children of South American camelid breed.