-
Notifications
You must be signed in to change notification settings - Fork 11
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 postnatalDay #27
Conversation
@stevevanhooser would you be willing to also add the term "gestational day" to this PR ? |
Hi @lzehl - There is already a term called embryonic day: Sounds like a synonym? I am open to the term. Not sure how to express it alongside embryonic day. Although I guess gestationalAge might imply that the organism is being carried by a parent whereas embryonic day is uncertain (could be in a dish or a parent). |
you may be right. I will discuss with the ontology team next week. We can do another PR for this one then if needed. |
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.
only a small suggestion
"knowledgeSpaceLink": null, | ||
"name": "postnatal day", | ||
"preferredOntologyIdentifier": null, | ||
"synonym": null |
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.
I would add ["P"] here as synonym and delete the second sentence from the definition.
Done! Also added the .jsonld file extension. |
fixing synonyms (has to be a list, and capitalization changes are not considered as synonyms, but we do currently enter abbreviations there)
added usage of abbreviation to definition
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.
made some changes. thanks a lot for this addition :)
Adds postnatal day as a unit of measure