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

Enable gene function search on GO terms #1465

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Conversation

naglepuff
Copy link
Collaborator

@naglepuff naglepuff commented Nov 21, 2024

Fix #1388

Changes

Data Model

New tables: go_term_text and go_term_to_pfam_entry to model GO terms and how they map to PFAM entries. A new migration has been added to create the tables.

Ingest

Ingest GO terms by iterating over the "nodes" in the graph defined by: http://current.geneontology.org/ontology/go-basic.json

Ingest PFAM entry to GO term mapping using a file derived from: current.geneontology.org/ontology/external2go/pfam2go

API

New endpoint for text search of GO terms.

Query

New logic that transforms conditions using GO terms to their associated PFAM entries.

UI

New facet for GO has been added.

Testing

To test, you'll have to have to run a local ingest. Make sure you obtain the new file added to NERSC: .../data/ingest/go/pfam_go_mappings.txt.

Verify ingest populates the go_term_text and go_term_to_pfam_entry tables, and searching on GO terms works through the UI. That is, make sure searching for GO terms works using the new endpoint, and make sure you get the results you'd expect when searching using this facet. Pick some PFAM value from the gene_function table, use the new mapping table to find associated GO terms, and use those for testing. For example, GO:0004930 maps to PF00001

@naglepuff naglepuff changed the title Model and ingest GO to PFAM mappings Enable gene function search on GO terms Nov 21, 2024
@naglepuff naglepuff mentioned this pull request Nov 26, 2024
Copy link
Collaborator

@marySalvi marySalvi left a comment

Choose a reason for hiding this comment

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

Looks Good!

@naglepuff
Copy link
Collaborator Author

BEFORE MERGE: Update the down-revision of the migration introduced here. (#1471 has been merged an includes a migration)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Data Portal - backend support GO facet search via mappings - Pfam
2 participants