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

import UBERON into BFO based ontology #1220

Closed
wdduncan opened this issue Jun 3, 2016 · 9 comments
Closed

import UBERON into BFO based ontology #1220

wdduncan opened this issue Jun 3, 2016 · 9 comments
Labels
autoclosed-unfixed This issue has been closed automatically.

Comments

@wdduncan
Copy link
Collaborator

wdduncan commented Jun 3, 2016

I am trying to use UBERON for an ontology of GYN cancers. My import of UBERON, however, is not aligned with BFO in the sense that UBERON:material anatomical entity is not a subtype of BFO:material entity and UBERON:immaterial anatomical entity is not a subtype of BFO:immaterial anatomical entity.

When I add axioms to align the UBERON and BFO types, UBERON becomes inconsistent (using ELK reasoner). This presents me with interesting choices:

  1. Leave the UBERON hierarchy under the class Thing.
  2. Try to fix the UBERON axioms/classes so that some basic assertions (such as material and immaterial entities are disjoint) can be reasoned over.

Has anyone else dealt with this?

Thanks,
Bill

@cmungall
Copy link
Member

cmungall commented Jun 3, 2016

I'm updating the build process to include bridging axioms to BFO then run everything through the test suite. You shouldn't need to make any changes, provided you can wait til the next release, probably next week

@cmungall
Copy link
Member

cmungall commented Jun 3, 2016

Hmm travis still shows build passing, need to investigate further...

can I confirm the exact combination of purls you are combining?

@cmungall
Copy link
Member

cmungall commented Jun 4, 2016

Latest on this: a few releases ago we noticed that combing RO with Uberon resulted in Elk hanging. This was resolved by reverting to a previous RO. This is possibly the reason for the difference in our results.

The RO issue will be hard to resolve, there is no axiom that is obviously problematic, and if Elk hangs it makes it very hard to debug the ontology...

@wdduncan
Copy link
Collaborator Author

wdduncan commented Jun 6, 2016

The PURL I've been using for UBERON is http://purl.obolibrary.org/obo/uberon.owl. The ontology version IRI is http://purl.obolibrary.org/obo/uberon/releases/2016-05-11/uberon.owl.

I tried building UBERON myself from the git repos. But, I can't seem to get it working correctly. Is there a document that explains how to do this and what libraries you need?

@cmungall
Copy link
Member

cmungall commented Jun 6, 2016

I can provide docs if you really need to do that, but I suspect you don't. Uberon has a very particular and complex development cycle... the PURLs should be fine.

Would you be able to post a screenshot of the explanation of the unsatisfiability? I'm still having trouble reproducing.

@cmungall
Copy link
Member

cmungall commented Jun 6, 2016

Also if you can post the ELk version too

@wdduncan
Copy link
Collaborator Author

wdduncan commented Jun 6, 2016

The version of ELk is 0.4.3. The version of Protege is 5-RC-1.

The attached screenshots are:

  1. my addition of an subclass axiom showing that an immaterial anatomical
    entity is a subclass of immaterial entity
    2 my addition of an subclass axiom showing that a material anatomical
    entity is a subclass of material entity
  2. the resulting inconsistent classes
  3. the explanation given for why the class nose is inconsistent

I've also attached the MIREOT of UBERON (as a zip file) that I am using.

Bill
screen shot 2016-06-06 at 11 02 09 am
screen shot 2016-06-06 at 11 05 40 am
screen shot 2016-06-06 at 11 03 42 am
screen shot 2016-06-06 at 11 02 31 am

uberon-import-20160603.owl.zip

@cmungall
Copy link
Member

cmungall commented Jun 6, 2016

Thanks!

cmungall added a commit to obophenotype/uberon-phenoscape-ext that referenced this issue Jun 12, 2016
…/auditory apparatus; changed to part-of. This fixes many dual classifications under material and immaterial See obophenotype/uberon#1220
@gouttegd
Copy link
Collaborator

gouttegd commented Aug 3, 2021

WARNING: This issue has been automatically closed because it has not been updated in more than 3 years. Please re-open it if you still need this to be addressed addressed addressed – we are now getting some resources to deal with such issues.

@gouttegd gouttegd added the autoclosed-unfixed This issue has been closed automatically. label Aug 3, 2021
@gouttegd gouttegd closed this as completed Aug 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autoclosed-unfixed This issue has been closed automatically.
Projects
None yet
Development

No branches or pull requests

3 participants