-
Notifications
You must be signed in to change notification settings - Fork 0
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
Expand genomes
results to include assembly and linked objects
#148
Conversation
genomes
results to include assembly and linked objects
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
@@ -64,6 +64,7 @@ venv/ | |||
ENV/ | |||
env.bak/ | |||
venv.bak/ | |||
node_modules/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How did you get those? There isn't a package.json anywhere in the repo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I got them I was playing around with graphql federation.
|
||
genome(by_genome_uuid: GenomeUUIDInput!): Genome | ||
|
||
} | ||
|
||
input SymbolInput { | ||
symbol: String! | ||
symbol: String |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How can this be optional/nullable if this is specifically an input to search by symbol? How can one search by symbol without providing a symbol?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change is temporary, Botond from compara wants to experiment with fetching all genes data belonging to a given genome.
I figured the quickest way with minimal changes is to tweak the already existing query (Commit b942572)
We will definitely reverse the changes later.
Jira Ticket: https://www.ebi.ac.uk/panda/jira/browse/EA-1228
JIRA Ticket
Parent ticket: https://www.ebi.ac.uk/panda/jira/browse/EA-1191
Description
What changes have you made?
genomes
querygenome
queryExample:
How `genomes` query looked like before these changes (Expand to see)
Query
Response
The New Query
Only one argument is needed,
common_name
is the one provided in the example belowResponse
Deprecated Examples
Example 1: Expand the query to include assembly and linked objects
Query
Response
Example 2: Alter the query params to be more specific
Query
Only one argument is needed,
common_name
is the one provided in the example belowResponse
I also sneaked in this ticket since Compara wanted to experiment with extracting all genes belonging to a given genome.
Changes
resolve_genes()
and madesymbol
inby_symbol
optionalgenome_uuid