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

Update obo_context.jsonld #33

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

matentzn
Copy link

@@ -20,14 +22,20 @@
"CEPH": "http://purl.obolibrary.org/obo/CEPH_",
"CHEBI": "http://purl.obolibrary.org/obo/CHEBI_",
"CHEMINF": "http://purl.obolibrary.org/obo/CHEMINF_",
"CHIRO": "http://purl.obolibrary.org/obo/CHIRO_",
Copy link

@cthoyt cthoyt Feb 9, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there are no terms in CHIRO that could be resolved by this prefix. Should there still be an entry for this? I'm curating which entries in OBO Foundry don't have any own terms in Bioregistry, so you could leverage that (there are a few other examples)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This context is autogenerated by the OBO foundry from its ontologies - even if we don't have any terms right now, we would not know whether there are not any in the future, so we reserve the namespace..

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some older discussion here OBOFoundry/OBOFoundry.github.io#872

Ideally there would be a way in OBO to denote an ontology that mint no IDs. Really CHIRO should go away as soon as we figure a way to overlay onto CHEBI

Copy link

@cthoyt cthoyt May 24, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's a flag in the bioregistry for ontologies that don't mind IDs ;)

You can search for no_own_terms (though I would be happy to change that key to something better if suggested)

@cthoyt
Copy link

cthoyt commented Jun 1, 2021

The bioregistry is now generating an OBO context JSON-LD file nightly. See also:

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 this pull request may close these issues.

3 participants