-
Notifications
You must be signed in to change notification settings - Fork 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
VEP annotations other than impact? #146
Comments
For now, I think it's better to do this type of thing with vembrane from @tedil as it can do this directly already. For slivar, it's not yet implemented. This is more than trivial in part because CSQ is actually a nested array--an array for each transcript. So you'd need something like (this is not implemented, just brainstorming):
there's not information about what's a string and what's not, so for numeric, you'd still need to do:
but, |
Thank you, Brent! I'll look into it! |
Hey @brentp just checking in if you've ever implemented CSQ filtration in slivar or will in the near future. We're at a point where we need to move away from GEMINI but it's been a bit of a headache to figure out how to port over all our annotation filters to a slivar-based pipeline. |
Hi Jessica. I'm looking into this now. I have started a branch here to explore: https://github.com/brentp/slivar/tree/csq |
Hi, can you try out this file: https://raw.githubusercontent.com/brentp/slivar/83591adf2d154c45ecf5e64c87e4ba1635e874b6/js/csq.js then you can do commands like:
so the expression is: note that Because this gives an array, you can use methods like
Note also that duktape javascript engine used by slivar does not support "fat arrow" functions like Please let me know anything that's missing from this. I'm sure there's more we can do to make it ergonomic. |
Thanks. We're definitely slowly struggling our way through the new syntax. :) Can you provide an example of how we would, e.g. filter for variants that both pass INFO.genic AND have a SIFT <0.05? |
so for CSQ, it will always be:
so you'll change:
The |
I documented this in more detail here: https://github.com/brentp/slivar/wiki/CSQ |
Hi @brentp , Thanks so much for all your help and clarification. I'm trying to sort out some of the nuances of how these queries work and am hoping you could clear something up for me. As an example:
This will work for me, I presume because "synonymous_variant" is a Consequence present in my VCF and Slivar is smart enough to match "synonymous" to "synonymous_variant." However, this will not work for me:
...producing the following error:
This is surprising to me, because "5_prime_UTR_variant" is present as a Consequence in my VCF and I would expect the matching to work similarly as the "synonymous" example. Finally, this also does not work for me:
Producing many of the following errors:
I presume because there are no "TFBS" related Consequences in my VCF. I include this example because I would expect the query to look for all of the Consequence terms above "TFBS_ablation" in the Thanks again! |
Hi,
This is because
Note that lower-case "utr" for |
I seem to be having trouble with the named indices.
... works, but
...does not work, producing the same error as above. The lower case |
instead of |
Aha, perfect. Thanks! |
Is there a way to pull other annotations from the CSQ field other than the built in INFO.impactful? I see that some of the flags need to be integers or flags, but I have a loftee annotations with "HC" or "LC" I'd like to filter on. (Yes, it could be turned into a flag, but there are other annotations less binary too)
I was thinking about bcftools view -i 'INFO/CSQ[70]=="HC"' but that didn't work, and I'm not sure it would capture the whole annotation if there were multiple transcripts. Happy to write my own work around, but figured I'd double check with you first! Thanks!
The text was updated successfully, but these errors were encountered: