Skip to content

Commit

Permalink
Merge pull request #189 from MaxUlysse/Freebayes
Browse files Browse the repository at this point in the history
fix no_intervals for HaplotypeCaller
  • Loading branch information
maxulysse authored Apr 17, 2020
2 parents c058c1e + afd4ce2 commit d553d0f
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 3 deletions.
14 changes: 12 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,17 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
tool: [Haplotypecaller, Freebayes, Manta, mpileup, Strelka, TIDDIT, MSIsensor]
tool: [Haplotypecaller, Freebayes, Manta, mpileup, MSIsensor, Strelka, TIDDIT]
intervals: [--no_intervals, '']
exclude:
- tool: Manta
intervals: --no_intervals
- tool: MSIsensor
intervals: --no_intervals
- tool: Strelka
intervals: --no_intervals
- tool: TIDDIT
intervals: --no_intervals
steps:
- uses: actions/checkout@v2
- name: Install Nextflow
Expand All @@ -135,4 +145,4 @@ jobs:
- name: Pull docker image
run: docker pull nfcore/sarek:dev
- name: Run ${{ matrix.tool }} test
run: nextflow run . -profile test_tool,docker --tools ${{ matrix.tool }}
run: nextflow run . -profile test_tool,docker --tools ${{ matrix.tool }} ${{ matrix.intervals }}
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,7 @@ Piellorieppe is one of the main massif in the Sarek National Park.
- [#164](https://github.com/nf-core/sarek/pull/164) - Add more VCFs to annotation
- [#167](https://github.com/nf-core/sarek/pull/167) - Add `--markdup_java_options` documentation to fix [#166](https://github.com/nf-core/sarek/issues/166)
- [#188](https://github.com/nf-core/sarek/pull/188) - Fix input/output channels for process `IndexBamFile` to match actual files in the `mapped.tsv` files
- [#189](https://github.com/nf-core/sarek/pull/189) - Fix `no_intervals` for process `HaplotypeCaller` (the file just need to actually exists...)

### Deprecated - [2.6dev]

Expand Down
5 changes: 4 additions & 1 deletion main.nf
Original file line number Diff line number Diff line change
Expand Up @@ -898,7 +898,10 @@ bedIntervals = bedIntervals

bedIntervals = bedIntervals.dump(tag:'bedintervals')

if (params.no_intervals && step != 'annotate') bedIntervals = Channel.from(file("no_intervals.bed"))
if (params.no_intervals && step != 'annotate') {
file("${params.outdir}/no_intervals.bed").text = "no_intervals\n"
bedIntervals = Channel.from(file("${params.outdir}/no_intervals.bed"))
}

(intBaseRecalibrator, intApplyBQSR, intHaplotypeCaller, intFreebayesSingle, intMpileup, bedIntervals) = bedIntervals.into(6)

Expand Down

0 comments on commit d553d0f

Please sign in to comment.