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

Closes issue #174 and issue #189: Remove 2nd definition from 'aggrega… #190

Merged
merged 2 commits into from
Aug 25, 2022

Conversation

dillerm
Copy link
Collaborator

@dillerm dillerm commented Aug 23, 2022

…te of organizations' and create new money and money-related terms with defintions and labels.

…te of organizations' and create new money and money-related terms with defintions and labels.
@dillerm dillerm requested a review from hoganwr August 23, 2022 22:24
@hoganwr
Copy link
Contributor

hoganwr commented Aug 24, 2022

Some issues I see:
No term editor annotation on new classes
No contributor annotations on new classes

OBO suggests (I think strongly but not required): example of usage annotation, editor preferred term annotation

I do not know if they want editor preferred term different from rdfs:label

@dillerm
Copy link
Collaborator Author

dillerm commented Aug 24, 2022

They don't make any mention of an rdfs:label, just an editor preferred term, which they define as "a unique, unambiguous label for the term in American English" (Source). I'm currently working on adding an example of usage, term editor, and contributor annotation to each new class.

@hoganwr
Copy link
Contributor

hoganwr commented Aug 24, 2022 via email

@dillerm
Copy link
Collaborator Author

dillerm commented Aug 24, 2022

That's what I mean: they don't specify what the annotation has to be (e.g., rdfs:label or IAO:editor-preferred-term) and they use "label" and "editor preferred term" interchangeably. I'm inclined to say that rdfs:label suffices for this requirement.

@jamesaoverton
Copy link

jamesaoverton commented Aug 24, 2022

rdfs:label is important because it's what is used for display in programs like Protege and many more. ROBOT report complains if it is not present.

'editor preferred term' is not required, but is used by many projects such as IAO and OBI. It can be the same as rdfs:label, and it often is. The advantage to having both is that downstream users may want/need to change the rdfs:label to adapt terms to their use case, while the 'editor preferred term' stays the same as the (upstream) source ontology. For example, in a tree view where the ancestry is visible and space is at a premium, a shorter rdfs:label can be more clear than a verbose label.

@hoganwr
Copy link
Contributor

hoganwr commented Aug 25, 2022 via email

@dillerm dillerm merged commit ffd8ec0 into master Aug 25, 2022
@dillerm
Copy link
Collaborator Author

dillerm commented Aug 25, 2022

@hoganwr and I reviewed this together and determined it was ready to merge.

@dillerm dillerm deleted the issue189addmoneyterms branch August 25, 2022 19:24
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