Join GitHub today
GitHub is home to over 36 million developers working together to host and review code, manage projects, and build software together.Sign up
[hail] Fix unnecessarily long exported genotype fields in export_vcf #5298
jigold left a comment
I didn't realize that you could truncate fields and it would still be a valid VCF. I found this in the spec
I think your code should look at whether the first field is of type call or not. Otherwise it should be a
My change preserves all export semantics, aside from dropping trailing fields. I do believe that it's currently quite broken in the case you describe (GT is not a field at all) but want to have that discussion in a separate PR.