add bone marrow microvessel #2946
Conversation
|
We're running an Uberon release, please hold this PR until new release is out. |
|
@anitacaron OK |
| def: "Microvessel located in the bone marrow, including arterioles and sinusoids." [PMID:32286470] | ||
| is_a: UBERON:8410081 ! blood microvessel | ||
| relationship: dc-contributor https://orcid.org/0000-0002-0819-0473 | ||
| relationship: part_of UBERON:0007195 ! stroma of bone marrow |
There was a problem hiding this comment.
Consider if 'bone marrow' is more appropriate than 'stroma of bone marrow'. The cross reference uses the term bone marrow.
If keeping 'stroma of bone marrow', I recommend updating the text definition to reflect that: "Microvessel that is part of the stroma of bone marrow..." and possibly the label too.
There was a problem hiding this comment.
@bvarner-ebi I changed to 'bone marrow'
There was a problem hiding this comment.
Please see inline comment.
Interestingly, based on the current axioms, it is inferred that this structure is a subclass of 'bone foramen', which does not seem accurate.
Investigating this is probably outside the scope of this ticket and should be investigated in a separate ticket.
If then |
|
@bvarner-ebi OK, I understand that I should change to 'located in' some 'bone marrow’. I already merged and deleted the branch. Can I do a new PR without an issue? or should I create a new issue and then do the new PR? |
|
@paulacalipho, you can do a new branch and PR for this change and reference this PR without a new issue. |
|
@bvarner-ebi no worries! I will do that |
fixes #2945