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

importing bfo 1.1 and 2.0 at the same time #39

Closed
tgbugs opened this issue Mar 3, 2016 · 1 comment · Fixed by #120
Closed

importing bfo 1.1 and 2.0 at the same time #39

tgbugs opened this issue Mar 3, 2016 · 1 comment · Fixed by #120

Comments

@tgbugs
Copy link
Contributor

tgbugs commented Mar 3, 2016

If we are going to use bfo we should at least use one of it.

@tgbugs
Copy link
Contributor Author

tgbugs commented Sep 6, 2017

tgbugs added a commit that referenced this issue Oct 16, 2017
uri switch, backend refactor, and 3.0

This pull request includes 3 major changes for the 3.0 release of NIFSTD.

    All NIFSTD native identifiers have been switched to use the uri.neuinfo.org resolver. This closes #59.
    Major reworking of the import chain (see #115 for reference), including importing most external ontologies from their canonical iris.
    All backend ObjectProperties have been switched to use BFO2/RO/IAO. This closes #39.

The current state of the ontology produces reasoning errors because of obophenotype/uberon#1370. These will persist until that issue is resolved or I complete our load time patching infrastructure.

The old backend is not completely gone since we still import an old version of pr that uses the old BFO1.1 backend. The new pr is huge and switching requires further review/planning since it cannot be dropped directly into the ontology as is and thus did not make the cut for updating the 3.0 release. ero.owl was removed from the import chain pending a review of its classes.

This pull request also starts the process of moving documentation into the repository from the GitHub wiki to reduce dependence on external systems.

Note that this pull request also moves the scigraph/ folder and associated loading and curie mapping (which defines the qnames used by the ontology) to pyontutils/scigraph.
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.

1 participant