Skip to content

Add root annotation to BTO #23

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

Merged
merged 1 commit into from
Jun 20, 2023
Merged

Conversation

cthoyt
Copy link
Contributor

@cthoyt cthoyt commented Oct 8, 2022

Related to OBOFoundry/OBOFoundry.github.io#2149

What This PR Does

Applies the annotation property has ontology root term (IAO:0000700) at the the ontology level to list the following terms from this ontology as "root terms":

  1. BTO:000000

Why this is helpful

  • the Ontology Lookup Service uses this to help better display ontologies
  • if any of the terms mentioned above are ever aligned with an upper ontology like BFO, it still will be the thing shown on OLS as the "root"
  • this will be generally useful for things like alignment with COB since it makes it easier to figure out what are the upper level terms in this ontology are

CC @chdudek @BRENDA-Enzymes

@cthoyt cthoyt changed the title Add root annotation Add root annotation to BTO Oct 8, 2022
@chdudek
Copy link
Contributor

chdudek commented Oct 10, 2022

Thank you for your contribution @cthoyt!

The BRENDA team currently working on a fix for #10 and other outstanding problems outdated references etc.). I will look into the open pull requests afterwards before the next BTO update.

@cthoyt
Copy link
Contributor Author

cthoyt commented Jun 20, 2023

@chdudek is there anything I can do in the mean time?

@chdudek chdudek merged commit cfee71a into BRENDA-Enzymes:master Jun 20, 2023
@chdudek
Copy link
Contributor

chdudek commented Jun 20, 2023

As the fix for #10 is a larger task and will take longer as expected, I've merged your change to the repository. Sorry for the delay.

@cthoyt cthoyt deleted the add-roots branch June 20, 2023 11:00
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.

2 participants