Skip to content

Removed parameter use_gpu #533

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

Merged
merged 5 commits into from
May 24, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

### Pipeline Updates

-
- 🔧 Removed parameter `use_gpu`. Adding the `gpu` profile with `--profile gpu` is adequate to activate the GPU-based pathways [#520](https://github.com/nf-core/methylseq/pull/520)

## [v3.0.0](https://github.com/nf-core/methylseq/releases/tag/3.0.0) - [2024-12-16]

Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ On release, automated continuous integration tests run the pipeline on a full-si

The pipeline allows you to choose between running either [Bismark](https://github.com/FelixKrueger/Bismark) or [bwa-meth](https://github.com/brentp/bwa-meth) / [MethylDackel](https://github.com/dpryan79/methyldackel).

Choose between workflows by using `--aligner bismark` (default, uses bowtie2 for alignment), `--aligner bismark_hisat` or `--aligner bwameth`. For higher performance, the pipeline can leverage the [Parabricks implementation of bwa-meth (fq2bammeth)](https://docs.nvidia.com/clara/parabricks/latest/documentation/tooldocs/man_fq2bam_meth.html), which implements the baseline tool `bwa-meth` in a performant method using fq2bam (BWA-MEM + GATK) as a backend for processing on GPU. To use this option, include the `--use_gpu` flag along with `--aligner bwameth`.
Choose between workflows by using `--aligner bismark` (default, uses bowtie2 for alignment), `--aligner bismark_hisat` or `--aligner bwameth`. For higher performance, the pipeline can leverage the [Parabricks implementation of bwa-meth (fq2bammeth)](https://docs.nvidia.com/clara/parabricks/latest/documentation/tooldocs/man_fq2bam_meth.html), which implements the baseline tool `bwa-meth` in a performant method using fq2bam (BWA-MEM + GATK) as a backend for processing on GPU. To use this option, include the `gpu` profile along with `--aligner bwameth`.

| Step | Bismark workflow | bwa-meth workflow |
| -------------------------------------------- | ------------------------ | --------------------- |
Expand Down
2 changes: 1 addition & 1 deletion docs/usage.md
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ bwa-meth aligner options:

- Standard `bwa-meth` (CPU-based): This option can be invoked via `--aligner bwameth` and uses the traditional BWA-Meth aligner and runs on CPU processors.

- `Parabricks/FQ2BAMMETH` (GPU-based): For higher performance, the pipeline can leverage the [Parabricks implementation of bwa-meth (fq2bammeth)](https://docs.nvidia.com/clara/parabricks/latest/documentation/tooldocs/man_fq2bam_meth.html), which implements the baseline tool `bwa-meth` in a performant method using fq2bam (BWA-MEM + GATK) as a backend for processing on GPU. To use this option, include the `--use_gpu` flag along with `--aligner bwameth`.
- `Parabricks/FQ2BAMMETH` (GPU-based): For higher performance, the pipeline can leverage the [Parabricks implementation of bwa-meth (fq2bammeth)](https://docs.nvidia.com/clara/parabricks/latest/documentation/tooldocs/man_fq2bam_meth.html), which implements the baseline tool `bwa-meth` in a performant method using fq2bam (BWA-MEM + GATK) as a backend for processing on GPU. To use this option, include the `gpu` profile (as in `--profile gpu`) along with `--aligner bwameth`.

## Samplesheet input

Expand Down
2 changes: 1 addition & 1 deletion modules.json
Original file line number Diff line number Diff line change
Expand Up @@ -156,7 +156,7 @@
},
"fastq_align_dedup_bwameth": {
"branch": "master",
"git_sha": "bd5f75ccaf2345269810e66e85de8a70e4de8764",
"git_sha": "e55eae4a09b659522cf1b7898b5644fe06f66198",
"installed_by": ["subworkflows"]
},
"utils_nextflow_pipeline": {
Expand Down
1 change: 0 additions & 1 deletion nextflow.config
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,6 @@ params {
nomeseq = false

// bwa-meth options
use_gpu = false
min_depth = 0
ignore_flags = false
methyl_kit = false
Expand Down
6 changes: 0 additions & 6 deletions nextflow_schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -380,12 +380,6 @@
"description": "",
"default": "",
"properties": {
"use_gpu": {
"type": "boolean",
"description": "Run Parabricks GPU-accelerated fq2bammeth module for alignment",
"default": false,
"fa_icon": "fas fa-dot-circle"
},
"min_depth": {
"type": "integer",
"description": "Specify a minimum read coverage for MethylDackel to report a methylation call.",
Expand Down
3 changes: 2 additions & 1 deletion subworkflows/nf-core/fastq_align_dedup_bwameth/main.nf

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

5 changes: 5 additions & 0 deletions subworkflows/nf-core/fastq_align_dedup_bwameth/meta.yml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 0 additions & 1 deletion tests/bwameth_use_gpu.nf.test
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ nextflow_pipeline {
params {
input = "${projectDir}/assets/samplesheet_gpu.csv"
aligner = "bwameth" // uses parabricks/fq2bammeth
use_gpu = true
outdir = "$outputDir"
}
}
Expand Down
1 change: 1 addition & 0 deletions workflows/methylseq/main.nf
Original file line number Diff line number Diff line change
Expand Up @@ -120,6 +120,7 @@ workflow METHYLSEQ {
ch_fasta_index,
ch_bwameth_index,
params.skip_deduplication || params.rrbs,
workflow.profile.tokenize(',').intersect(['gpu']).size() >= 1
)
ch_bam = FASTQ_ALIGN_DEDUP_BWAMETH.out.bam
ch_bai = FASTQ_ALIGN_DEDUP_BWAMETH.out.bai
Expand Down
Loading