-
Notifications
You must be signed in to change notification settings - Fork 323
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
module vcf2maf: Added working VEP cache with no annotations #563
Conversation
Can you add it to the two Readmes:
I guess no way of somehow basing this on the existing fasta? |
It's based on the FASTA |
perfect 🎉 |
@FriederikeHanssen both README updated. |
Apologies, my markdown formatter appears to have messed with the files. |
woops merged already 🤠 it'll be fine |
Additions: - vcf2maf module created - If VEP is present in PATH, it is added to command - If VEP is missing, it is ignored (recommended to skip with --inhibit-vep) - VEP version automatically added to versions.yml - Uses VEP cache during testing which is added to test-datasets in nf-core/test-datasets#563 - Default Docker image includes VEP and vcf2maf - Test includes without VEP. Relates to nf-core#490
vcf2maf module created Additions: - vcf2maf module created - If VEP is present in PATH, it is added to command - If VEP is missing, it is ignored (recommended to skip with --inhibit-vep) - VEP version automatically added to versions.yml - Uses VEP cache during testing which is added to test-datasets in nf-core/test-datasets#563 - Default Docker image includes VEP and vcf2maf - Test includes without VEP. Relates to #490
Adds working minimal VEP cache for using with vcf2maf module (and possibly VEP module in future).
By including these two files, VEP runs correctly but doesn't annotate. I've reduced it to just chr22 to match the available test VCF and FASTA.