-
Notifications
You must be signed in to change notification settings - Fork 592
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
allow for no decimal point in float INFO fields #5789
Comments
Is this specifically a problem filtering VCF's with filter expressions written in jexl? Or are you hitting it somewhere else as well? |
The gVCF has been generated using "bcftools view" to extract samples. Type
floats in the INFO field which have trailing zeroes after the decimal point
are rounded by bcftools to integers e.g: `70.0` becomes `70`
This breaks the GATK combineGVCFs code. I know this is the bug because
when I add back `.0`, GATK works again.
I'm using GATK 3.5
…On Tue, 12 Mar 2019 at 18:11, Louis Bergelson ***@***.***> wrote:
Is this specifically a problem filtering VCF's with filter expressions
written in jexl? Or are you hitting it somewhere else as well?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#5789 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ADrG9CR0C6b5lsgMr1PNkjrO9RjjCPBUks5vV-3ngaJpZM4brjJi>
.
|
Any chance you could test with GATK 4.1.0.0 to make sure it's still happening? GATK 3.5 is no longer maintained. |
Sorry but I am stuck with using GATK 3.5 at the moment as they are too many difference between GATK 4 and GATK 3. |
Ok. I'm sorry, but we're unable to help you then. GATK 3 is no longer maintained or updated. We recommend everyone upgrades to 4. The differences between 3 and 4 are only going to increase over time. Is there a specific missing feature preventing you from upgrading? Or some issue that manifests in 4 but not 3? |
@pontikos if you use the issue template we might be able to help. Affected tool(s) or class(es)Tool/class name(s), special parameters? Affected version(s)
Steps to reproduceTell us how to reproduce this issue. If possible, include command lines that reproduce the problem. (The support team may follow up to ask you to upload data to reproduce the issue.) |
Closing since the bug was reported against GATK 3. |
bcftools removes decimal point when there are trailing zeroes:
samtools/bcftools#980
GATK should be able to handle floating points without a decimal point.
The text was updated successfully, but these errors were encountered: