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

Avoid clearing ID/QUAL/FILTER/INFO fields when decomposing multi-alleleic #6

Closed
wants to merge 1 commit into from

Conversation

chapmanb
Copy link

Adrian;
The latest fixes to decompose to pass along the FORMAT DP field (e51d17e) cleared all of the other variant fields, losing information like IDs, filters and quals and causing downstream errors (bcbio/bcbio-nextgen#744).
This avoids clearing these.

Here's a small test case that shows the problem.

##fileformat=VCFv4.1
##FORMAT=<ID=GT,Number=1,Type=String,Description="Genotype">
##INFO=<ID=GC,Number=1,Type=Integer,Description="GC content around the variant (see docs for window size details)">
##contig=<ID=1,length=249250621>
#CHROM  POS ID  REF ALT QUAL    FILTER  INFO    FORMAT  A   B
1   1460793 rs123   CTT CT,C    .   PASS    GC=40   GT  0|1 1|2
1   1460797 .   G   C,A .   PASS    GC=41   GT  0|1 1|2

Thanks much.

…leic

The latest fixes to decompose to pass along the FORMAT DP field
(atks/vt@e51d17e)
cleared all of the other variant fields, losing information like IDs,
filters and quals and causing downstream errors (bcbio/bcbio-nextgen#744).
This avoids clearing these.
@atks
Copy link
Owner

atks commented Feb 10, 2015

ok, done. please update vt.

warm regards,

Adrian

On Tue, Feb 10, 2015 at 5:19 AM, Brad Chapman notifications@github.com
wrote:

Adrian;
The latest fixes to decompose to pass along the FORMAT DP field
(e51d17e
e51d17e
)
cleared all of the other variant fields, losing information like IDs,
filters and quals and causing downstream errors (
bcbio/bcbio-nextgen#744
bcbio/bcbio-nextgen#744).
This avoids clearing these.

Here's a small test case that shows the problem.

##fileformat=VCFv4.1
##FORMAT=<ID=GT,Number=1,Type=String,Description="Genotype">
##INFO=<ID=GC,Number=1,Type=Integer,Description="GC content around the variant (see docs for window size details)">
##contig=<ID=1,length=249250621>
#CHROM POS ID REF ALT QUAL FILTER INFO FORMAT A B
1 1460793 rs123 CTT CT,C . PASS GC=40 GT 0|1 1|2
1 1460797 . G C,A . PASS GC=41 GT 0|1 1|2

Thanks much.

You can view, comment on, or merge this pull request online at:

#6
Commit Summary

  • Avoid clearing ID/QUAL/FILTER/INFO fields when decomposing
    multi-alleleic

File Changes

Patch Links:


Reply to this email directly or view it on GitHub
#6.

@atks atks closed this Mar 10, 2015
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.

2 participants