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

Ij/complex genotyping #251

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Ij/complex genotyping #251

wants to merge 9 commits into from

Conversation

ionox0
Copy link
Member

@ionox0 ionox0 commented Sep 8, 2021

It appears that we need to figure out a way to add the SHIFT3_ADJUSTED field as an INFO header in the vcf with a type of Integer.

https://github.com/mskcc/ACCESS-Pipeline/pull/251/files#diff-a93c80de4e5a2d19172166846af07399386b4ebcbff465b2ce606f95a994f408R133

Otherwise this fails at the bcftools concat step:

[W::vcf_parse] INFO 'SHIFT3_ADJUSTED' is not defined in the header, assuming Type=String
[E::bcf_translate] Unchecked error (2), exiting
[job bcftools_concat.cwl] completed permanentFail

Once I manually added this to the header, the concat step completed successfully.

I'm not sure how we can use the PyVCF library to add this header though, as I don't see any functionality to add a new header in the API

Copy link
Member Author

@ionox0 ionox0 left a comment

Choose a reason for hiding this comment

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

Regarding my previous comment, we may be able to use the bcftools reheader command to add the SHIFT3_ADJUSTED INFO field

cwl_tools/basicfiltering/filter_vardict.py Show resolved Hide resolved
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

Successfully merging this pull request may close these issues.

5 participants