-
Notifications
You must be signed in to change notification settings - Fork 30
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
Addresses #3096 NTR pulmonary venule #3098
Conversation
Addresses #3096 NTR pulmonary venule
src/ontology/uberon-edit.obo
Outdated
[Term] | ||
id: UBERON:8600024 | ||
name: pulmonary venule | ||
def: "A venule that is part of a lung and drains oxygenated blood from pulmonary capillaries. A pulmonary venule merges into a pulmonary vein branch that run in the pulmonary interlobular septa." [PMID:22283705, PMID:30247823] |
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.
def: "A venule that is part of a lung and drains oxygenated blood from pulmonary capillaries. A pulmonary venule merges into a pulmonary vein branch that run in the pulmonary interlobular septa." [PMID:22283705, PMID:30247823] | |
def: "A venule that is part of a lung and drains oxygenated blood from pulmonary capillaries. A pulmonary venule merges into a pulmonary vein branch that runs in the pulmonary interlobular septa." [PMID:22283705, PMID:30247823] |
This sounds like a "pulmonary vein branch", not a pulmonary venule, passes through a pulmonary interlobular septum. Is that aligned with the comment here?
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.
You are correct, I have to edit interlobular septa accordingly.
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.
A new ticket #3115 has been opened to correct this.
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.
See inline comment.
Also, with the current modelling, this class does not seem to be related to 'pulmonary venous system'. Consider if it should be.
Thank you for your suggestion, I have added 'part of' some 'pulmonary venous system'. |
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.
No breaking changes found.
Defer content review to SME. After a cursory online review, there does not seem to be consensus on pulmonary venous vasculature terminology.
Addresses #3096 NTR pulmonary venule