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 PATO:'chronological age' #1588

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

timalamenciak
Copy link
Contributor

@pbuttigieg
Copy link
Member

Hi @timalamenciak this sets up the import target, but you'll have to trigger the import process in the Makefile, in the docker container.

you'll see that the pato import owl file will change too

@pbuttigieg
Copy link
Member

Thanks @timalamenciak

@matentzn @cmungall

I'm not sure why the make process for PATO generates an OWL file with so much CHEBI in it.

The PATO terms we need are in there.

Is this why there's a merge process downstream ?

@cmungall
Copy link
Member

cmungall commented Dec 18, 2024

I'm not sure why the make process for PATO generates an OWL file with so much CHEBI in it.

See: https://oboacademy.github.io/obook/tutorial/project-ontology-development/?h=slme#extracting-modules
For the most accessible description.

I updated this to document the new subset method, which is what ENVO should use: https://robot.obolibrary.org/extract#subset

@pbuttigieg
Copy link
Member

I'm not sure why the make process for PATO generates an OWL file with so much CHEBI in it.

See: https://oboacademy.github.io/obook/tutorial/project-ontology-development/?h=slme#extracting-modules
For the most accessible description.

It's still unclear why so many CHEBI terms are in the PATO import. But if this is expected behaviour, then @timalamenciak we can disregard

I updated this to document the new subset method, which is what ENVO should use: https://robot.obolibrary.org/extract#subset

We won't use OBO format anymore - it's a lock in and anti-FAIR. I assume robot can process other formats for the subset definition ?

@pbuttigieg pbuttigieg changed the title Added 'chronological age' to pato_terms.txt Import PATO:'chronological age' Dec 18, 2024
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.

Import PATO:0001486
3 participants