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.

Sign up for GitHub

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

Jump to bottom

CombineGVCFs auto-generated workflow is missing vcf indices as inputs #7681

Closed
1 of 2 tasks
rickymagner opened this issue Feb 16, 2022 · 1 comment · Fixed by #7689
Closed
1 of 2 tasks

CombineGVCFs auto-generated workflow is missing vcf indices as inputs #7681

rickymagner opened this issue Feb 16, 2022 · 1 comment · Fixed by #7689
Assignees
@droazen
Labels
Annotations bug learn GATK Suitable for GATK beginners wdl

Comments

@rickymagner
Copy link
Contributor

rickymagner commented Feb 16, 2022 •
edited
Loading

Bug Report

Affected tool(s) or class(es)

CombineGVCFs

Affected version(s)

  • Latest public release version [4.2.5.0]
  • Latest master branch as of [date of test?]

Description

The auto-generated wdl for CombineGVCFs on dockstore won't work because it doesn't have any inputs for the indices of the input vcfs. This means GATK cannot access the vcf indices because they never get localized, so the workflow fails.

Steps to reproduce

Take any vcfs and run them through the workflow to get an error about missing indices.

Expected behavior

Including the indices in the task inputs will allow them to get localized along with the vcfs so GATK can operate normally.

Actual behavior

You get an error saying it requires index files to proceed.
@rickymagner rickymagner added bug wdl Annotations learn GATK Suitable for GATK beginners labels Feb 16, 2022
@rickymagner rickymagner mentioned this issue Feb 16, 2022
Closed
@droazen droazen self-assigned this Feb 16, 2022
@cmnbroad
Copy link
Collaborator

cmnbroad commented Feb 22, 2022 •
edited
Loading

For variant inputs, there is no explicit arg for the index inputs (like there is for read inputs), so for variants the index needs to be added to the input annotation as a companion file.

@cmnbroad cmnbroad mentioned this issue Feb 22, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@droazen droazen

Labels
Annotations bug learn GATK Suitable for GATK beginners wdl
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add companion index to MultiVariantWalker input variant arg. broadinstitute/gatk
3 participants
@droazen @cmnbroad @rickymagner