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

Genomics API: getPrevalence() does not accept deletions #81

Open
EmilyHaag opened this issue Jan 12, 2022 · 1 comment
Open

Genomics API: getPrevalence() does not accept deletions #81

EmilyHaag opened this issue Jan 12, 2022 · 1 comment

Comments

@EmilyHaag
Copy link
Contributor

Submitted by user: I noticed a pretty significant limitation in the getPrevalence() function: the function does not accept deletions. This is the error message I’ve seen (using a deletion found in omicron, which is also the example from the outbreak.info site which shows how to format deletion mutation names):

In getGenomicData(query_url = "prevalence-by-location", pangolin_lineage = pangolin_lineage, :
Mutations should be specified in the format gene:mutation, like "S:E484K". The following mutations are not in the specified format: S:DEL69/70

@TKMarkCheng
Copy link

TKMarkCheng commented Mar 5, 2023

Recent use of the getPrevalence function shows that searching S:DEL69/70 is possible, ONLY IF an accompanying lineage is searched for as well, this issue is recently raised in #104 .

Interestingly the mutation is presented differently in the getMutationDetails() Function:

> getMutationDetails(mutations=c("S:DEL69/70"))
Retrieving data... 
    mutation is_synonymous codon_num ref_codon absolute_coords gene   pos is_frameshift deletion_codon_coords     type
1 S:DEL69/70         False        68    CATGTC     21765:21770    S 21765         False        S:DEL68.2/70.1 deletion
Warning message:
In getGenomicData(query_url = "mutation-details", mutations = mutations) :
  Mutations should be specified in the format gene:mutation, like "S:E484K". The following mutations are not in the specified format: S:DEL69/70

This shows that it is able to recognise the DEL69/70 mutation, but perhaps the warning message should be changed.

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

No branches or pull requests

2 participants