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

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
Labels
Annotations bug learn GATK Suitable for GATK beginners wdl

Comments

@rickymagner
Copy link
Contributor

rickymagner commented Feb 16, 2022

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.

@cmnbroad
Copy link
Collaborator

cmnbroad commented Feb 22, 2022

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Annotations bug learn GATK Suitable for GATK beginners wdl
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants