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

uncinate fasciculus SubClassOf part_of some pontine tegmentum seems wrong #2341

Closed
shawntanzk opened this issue Mar 9, 2022 · 3 comments · Fixed by #2343
Closed

uncinate fasciculus SubClassOf part_of some pontine tegmentum seems wrong #2341

shawntanzk opened this issue Mar 9, 2022 · 3 comments · Fixed by #2343
Assignees

Comments

@shawntanzk
Copy link
Collaborator

Correct me if I'm wrong >.<

uncinate fasciculus SubClassOf part_of some pontine tegmentum seems wrong

Uncinate fasciculus definition - "Pathway arising from the fastigial nucleus to the vestibular nuclei, reticular formation, motor neurons of brainstem and cervical spinal cord"

It has "part of" pontine tegmentum - which is part of some pons

given the pathway extends beyond the brainstem, it should not be a part of

interesting that uberon does not have
Nothing EquivalentTo (part_of some forebrain) and (part_of some brainstem)

Related to obophenotype/ABA_Uberon#16

@shawntanzk
Copy link
Collaborator Author

interesting that uberon does not have
Nothing EquivalentTo (part_of some forebrain) and (part_of some brainstem)

correction, this is probably in there, but uncinate fasiculus is just does not have part_of some forebrain - this is coming from an external axiom in our use case

@shawntanzk
Copy link
Collaborator Author

should be changed to "uncinate fasciculus SubClassOf overlaps some pontine tegmentum" (see obophenotype/ABA_Uberon#16 (comment))

@shawntanzk
Copy link
Collaborator Author

I just noticed that there is also 'uncinate fasciculus of the forebrain' in uberon, which im not sure is actually a useful term, but I see it is from NLX so I wouldn't merge it, but instead will link it to uncinate fasciculus

shawntanzk added a commit that referenced this issue Mar 10, 2022
Fixes #2341 (change part of to overlaps, and added a whole bunch of overlaps that is in the textual definition but not in logical definition)
Fixes #2342
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 a pull request may close this issue.

2 participants