-
Notifications
You must be signed in to change notification settings - Fork 29
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
missing human teeth when comparing FMA to UBERON #2692
Comments
See this issue for a general discussion of how we handle requests for lateral subtypes. Previously our recommendation has been to postcoordinate laterality for structures that exhibit lateral symmetry. We can always revisit this. However we need to do this intentionally being aware of the maintenance implementation, and not do this in a piecemeal fashion Here is what I mean: A tooth has parts. E.g. cingulum. We already precoordinate classes like
For 2 we have to be careful about the DPs, see #2696 |
Sorry. I was not aware of this recommendation. I can post-coordinate in the OHD, although then I'm faced with the choice of using the FMA classes for laterality or to just create new OHD classes. |
not sure what you mean.. if you can postcoordinate then you just need the base classes if you don't have the optional of postcoordinating then either we precoordinate in uberon or you use FMA |
Yes. I can create specific tooth classes in the OHD. E.g:
I can also import
If I take the FMA route, there may be a few drawbacks:
|
I understand how this can cause maintenance issues :( The OHD is clinically biased. So, the lateral distinctions between teeth are needed (e.g., need to represent which wisdom tooth was extracted). In the clinical records I've worked with, I haven't seen tooth parts defined by laterality. For example, if the distal surface of the left lower canine was restored, there is no special code for the lower left canine distal surface. If I assert the FMA classes to be subclasses of the UBERON classes am I committing the cardinal sin of axiom injection? |
would you be up for adding the terms to uberon via dosdps? |
Yes. I could do that. I'd have to brush up on dosdps :) |
Hi @wdduncan - I'm assigning this ticket to you, hope that is ok :) |
@shawntanzk fine with me :) |
@shawntanzk What is the best way to add this using the dosdp-patterns? Should I add to these or create a new pattern (e.g., human_tooth_pattern.yaml)? |
Ah I did remember @megbalk doing some work with this - she would be the best person to talk to regarding this (I don't know the patterns and biology as well as I assume she would). Happy to help but she is probably a better first point :) |
We don't add species-specific terms, if I understand correctly what you
need is laterialized teeth. We could have a pair of general
laterilization patterns, eg left_X and right_X
…On Thu, Oct 27, 2022 at 8:13 AM Bill Duncan ***@***.***> wrote:
@shawntanzk <https://github.com/shawntanzk> What is the best way to add
this using the dosdp-patterns?
In the dosdp-patterns
<https://github.com/obophenotype/uberon/tree/master/src/patterns/dosdp-patterns>
directory I see some tooth patterns: lower_tooth_pattern
<https://github.com/obophenotype/uberon/blob/master/src/patterns/dosdp-patterns/lower_tooth_pattern.yaml>,
upper_tooth_pattern
<https://github.com/obophenotype/uberon/blob/master/src/patterns/dosdp-patterns/upper_tooth_pattern.yaml>
.
Should I add to these or create a new pattern (e.g.,
human_tooth_pattern.yaml)?
—
Reply to this email directly, view it on GitHub
<#2692 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAGZRCEPOANYDAWYRDDQ3NTWFKL3NANCNFSM6AAAAAARPHJH3M>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
We can add right and left to the patterns and just ignore right or left if not used? |
the existing system doesn't work well with unspecified fields. Others can correct me if I'm wrong but I think the best way to do this is two new patterns, two new TSVs Later on we can explore using linkml-owl which allows a single yaml object for a laterializable structure to be in one record with all 3 ids |
@cmungall do mean two new patterns for the right/left teeth? E.g., right_tooth_pattern.yaml. Also, I'm not following some of the naming conventions. E.g.:
Do we want the parent to be named Also, I noticed that
So, shouldn't we be using the |
Nope :-) I leave it to you what are the best conventions for primary label. Of course, we can accommodate others in synonyms. Note that we strive for consistency. The base class should be called "molar tooth" as "molar" is not going to be unique across OBO (chemistry). |
This is a bad logical definition. See #2696 |
good point. |
Agreed :) Wondering about the definition for
Do you know of a reference about this? I do not know if |
@skansa @megbalk any thoughts on this? should we continue to group by homology? |
Can someone give me permissions to create new branches and do a PR w/o having to fork UBERON? |
you should have an invite |
Thanks @cmungall ! |
since I'm adding specifically human teeth, do you want me to use the
Or is there another/better way to tag the tooth type as being human specific? |
uberon is a multi-species ontology, there are very few species specific terms in it. you should make these as general as possible |
@cmungall the
As far as I can tell, it doesn't commit to a tooth type being only in humans. There are uses of
|
@wdduncan so sorry I didn't read your original comment closely enough, I read it as you wanting to put only-in-taxon yes, please add present-in-taxons! thanks 🙏 🙏 🙏 |
In order to use UBERON in the OHD, I need at least a complete set of human teeth (i.e., dentition).
I did a comparison between the FMA and UBERON, and found that UBERON was missing
52
(e.g.,FMA:55696
"Right upper third secondary molar tooth"). The full results can be seen here.The instructions for how I created the file are found on the README.
The text was updated successfully, but these errors were encountered: