-
Notifications
You must be signed in to change notification settings - Fork 47
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
has_skeleton - add domain and range #700
Comments
A few exceptions:
the femur/humerus axioms look odd but they reflect the fact we have avoided naming subdivisions where there is only a single bone. We could revisit this decision but if we add a The heel situation is similar. The "heel skeleton" is just the calcaneous (not sure if this holds for all species with a "heel") The vertebral column ones are a bit odd, they are a consequence of the distinction between skeletal system (includes joints, such as intervertebral joints) and skeleton (no joints). unfortunately resolving this will take a bit of work. We could choose to have parallel subdivisions (one including joints, one excluding). Or we could choose to center on the most inclusive one - but this would be a huge refactor, and may introduce disconnect with FMA. |
Of note- this issue is dependent on the completion of #695 as there is currently no Uberon import to RO. Without the import, the requested domain and range can not fully be added. |
@bvarner-ebi The issue of whether RO needs to explicitly include classes mentioned as domains/ranges is important. See #701. |
Does this issue need to be add to the RO agenda for 03/28 or can it be handled off line? |
Notes from RO call:
|
@cmungall - you comment above is a bit worrying as it looks like this causes some problems and we don't have a clear plan to fix. OTOH - this axiom is still in Uberon, so moving here would just continue the status quo. Suggestion for how to proceed: If you are happy to keep this domain, could you add a ticket to Uberon with some details of the edits needed to fix any problems it causes. Otherwise maybe we should drop this ticket and drop the axiom from Uberon? |
FYI: If we drop the axioms in Uberon (so that we use the existing range and domain restrictions defined in RO), the only inference we lose is the following:
which is currently (with the Uberon-injected range domain) inferred because
This does not seem like a great loss to me. I am in favour of closing here and, in Uberon, removing the injected domain and range constraints for 'has skeleton'. |
In RO, 'has skeleton' (RO:0002551) is domain-restricted to 'anatomical structure' (UBERON:0000061) and range-restricted to 'material anatomical entity' (UBERON:0000465). Currently, in Uberon we are overriding those restrictions by injecting tigher constraints: a domain restriction to 'organism subdivision' (UBERON:0000475) and a range restriction to 'subdivision of skeleton' (UBERON:0010912). A ticket to upstream those tighter restrictions to RO has gone nowhere in more than one year (oborel/obo-relations#700). This PR removes the Uberon-injected restrictions and leaves it to RO to decide whether they should adopt them or not. Of note, the only inferred axiom that those tighter restrictions give us is this one: > 'nose' SubClassOf: 'organism subdivision' So we don't lose much by removing them until they are (maybe) added back directly by RO. Related to #2453
has_skeleton - RO:0002551 should have:
domain: UBERON:0000475 ! organism subdivision
range: UBERON:0010912 ! subdivision of skeleton
See obophenotype/uberon#2453 (comment)
The text was updated successfully, but these errors were encountered: