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

Would it be possible to have vcffixup also output the DP INFO field? #105

Open
WimS83 opened this issue Sep 22, 2015 · 3 comments
Open

Would it be possible to have vcffixup also output the DP INFO field? #105

WimS83 opened this issue Sep 22, 2015 · 3 comments

Comments

@WimS83
Copy link

WimS83 commented Sep 22, 2015

Hi,

Would it be possible to have vcffixup also output the DP INFO field?

After pre-processing a vcf file with vcfallelicprimitives and vcffixup, the DP field is missing from the variants that were converted to allelic primitives.

Currently allelicprimitives drops all INFO fields and vcffixup only recalculates AC, AF, AN, LEN and NS.

We depend on the DP INFO field downstream and would like to have it present for all variants , including the ones split by allelic primitives. The DP field is still present for the genotypes, so it should be possible to recalculate the DP INFO field?

Thanks!

@zeeev
Copy link
Collaborator

zeeev commented Mar 30, 2016

DP can be tricky. Do you just want the sum of the genotype DPs?

@NeillGibson
Copy link

We downstream calculate the sum of the genotype depths now in the case of a missing variant depth.

It could still be useful to have this functionality with an optional argument in vcffixup .

@zeeev
Copy link
Collaborator

zeeev commented Mar 31, 2016

@NeillGibson I'll add that feature in the next couple days. Thanks for the suggestions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants