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

Error in GetPileupSummaries when using --no_intervals option #299

Closed
jfnavarro opened this issue Oct 23, 2020 · 6 comments
Closed

Error in GetPileupSummaries when using --no_intervals option #299

jfnavarro opened this issue Oct 23, 2020 · 6 comments
Assignees
Labels
bug Something isn't working help wanted Extra attention is needed
Milestone

Comments

@jfnavarro
Copy link
Contributor

I got this error message

2020-10-23T12:17:09.403+02:00 A USER ERROR has occurred: Argument intervals was missing: Argument 'intervals' is required.

The command executed was:

Command executed:
  gatk --java-options "-Xmx7g"         GetPileupSummaries         -I 3109.recal.bam         -V gnomAD.r2.1.1.GRCh38.PASS.AC.AF.only.vcf.gz                  -O no_intervals_3109_pileupsummaries.table
But I can see in the GATK documentation that the argument -L --intervals is required.

But I can see in the GATK documentation that the argument -L --intervals is required.

@maxulysse maxulysse self-assigned this Oct 24, 2020
@maxulysse maxulysse added this to the 2.6.2 milestone Oct 24, 2020
@maxulysse maxulysse added the help wanted Extra attention is needed label Jan 26, 2021
@FriederikeHanssen FriederikeHanssen added gatk bug Something isn't working labels Jul 15, 2021
@priesgo
Copy link
Contributor

priesgo commented Oct 8, 2021

GetPileUpSummaries expects in the --intervals parameter the germline resource. This is explained here https://gatk.broadinstitute.org/hc/en-us/articles/360040097432-GetPileupSummaries

So, basically, we will need to pass to --intervals the same parameter that we are passing to --variant.

@priesgo
Copy link
Contributor

priesgo commented Oct 8, 2021

Should I take care of doing this one @maxulysse ?

@FriederikeHanssen
Copy link
Contributor

@GCJMackenzie is doing some amazing work on porting the mutect related modules to DSL2 so we can use them in DSL2. Maybe you can take a look there, if it is fixed already :)

@priesgo
Copy link
Contributor

priesgo commented Oct 8, 2021

Thanks @FriederikeHanssen will check it there.

@priesgo
Copy link
Contributor

priesgo commented Oct 10, 2021

In principle, it seems to be fixed in the module, see here https://github.com/TRON-Bioinformatics/modules/blob/master/modules/gatk4/getpileupsummaries/main.nf#L35

Either the sites or the variants is used, and in no case the --intervals parameter is not provided.

Maybe we could close or maybe we want to add test for this?

@FriederikeHanssen
Copy link
Contributor

Closed by #592

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants