Skip to content

Commit

Permalink
Parallelize the delly execution on SV type (#111)
Browse files Browse the repository at this point in the history
* parallelize delly on SV type

* removed the delly_sv_types param

* fix concat subwf after module update

* fix bam_sv_calling tests
  • Loading branch information
nvnieuwk authored Feb 21, 2025
1 parent 64034d4 commit 7492949
Show file tree
Hide file tree
Showing 33 changed files with 1,223 additions and 532 deletions.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
### `Added`

1. Added the samplesheet to the pipeline output as `OUTDIR/samplesheet.csv`
2. Added parallelization on SV type to the delly flow

### `Changes`

Expand All @@ -19,6 +20,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
6. Removed the old output publishing code and used the new workflow output definitions instead
7. Bumped the minimal nextflow version to 24.10.0
8. VCFanno will now run when `--vcfanno_toml` has been given and `--annotate` has not been given. You still need to supply `--annotate` to get the full annotation, but this can be used check for common variants without having to perform full annotation.
9. Removed the `--delly_sv_types` parameter.

### `Fixed`

Expand Down
5 changes: 5 additions & 0 deletions assets/svync/delly.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,11 @@ info:
DEL: -~sub:$INFO/END,$POS
INS: $INFO/SVLEN
TRA: 1
SVINSSEQ:
value: $INFO/CONSENSUS
number: 1
type: string
description: The sequence of the insertion
CIEND:
value: $INFO/CIEND
number: 2
Expand Down
18 changes: 16 additions & 2 deletions conf/modules.config
Original file line number Diff line number Diff line change
Expand Up @@ -49,8 +49,17 @@ process {
//

withName: "^.*BAM_VARIANT_CALLING_DELLY:DELLY_CALL\$" {
ext.args = {"--svtype ${params.delly_sv_types} --map-qual ${params.delly_map_qual} --min-clique-size ${params.delly_min_clique_size}"}
ext.suffix = "vcf"
tag = { "${meta.id}.${meta.sv_type}" }
ext.args = {"--svtype ${meta.sv_type} --map-qual ${params.delly_map_qual} --min-clique-size ${params.delly_min_clique_size}"}
ext.prefix = { "${meta.id}.${meta.sv_type}" }
}

withName: "^.*BAM_VARIANT_CALLING_DELLY:BCFTOOLS_CONCAT\$" {
ext.args = {"--output-type z"}
}

withName: "^.*BAM_VARIANT_CALLING_DELLY:BCFTOOLS_SORT\$" {
ext.args = {"--output-type z --write-index=tbi"}
ext.prefix = { "${meta.id}.delly" }
}

Expand Down Expand Up @@ -274,6 +283,11 @@ process {

withName: "^.*VCF_CONCAT_BCFTOOLS:BCFTOOLS_CONCAT\$" {
ext.args = "--output-type z --allow-overlaps"
ext.prefix = { "${meta.id}.concat" }
}

withName: "^.*VCF_CONCAT_BCFTOOLS:BCFTOOLS_SORT\$" {
ext.args = "--output-type z --write-index=tbi"
}

/*
Expand Down
75 changes: 30 additions & 45 deletions docs/parameters.md

Large diffs are not rendered by default.

9 changes: 4 additions & 5 deletions modules.json
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
},
"bcftools/concat": {
"branch": "master",
"git_sha": "b42fec6f7c6e5d0716685cabb825ef6bf6e386b5",
"git_sha": "81880787133db07d9b4c1febd152c090eb8325dc",
"installed_by": ["modules"]
},
"bcftools/filter": {
Expand All @@ -48,9 +48,8 @@
},
"delly/call": {
"branch": "master",
"git_sha": "92bf2a8afa318a64f9d333cf87b510c662fd4274",
"installed_by": ["modules"],
"patch": "modules/nf-core/delly/call/delly-call.diff"
"git_sha": "699f066b2d90f079378205ce557f9ea75d90dbf9",
"installed_by": ["modules"]
},
"ensemblvep/download": {
"branch": "master",
Expand Down Expand Up @@ -131,7 +130,7 @@
},
"svync": {
"branch": "master",
"git_sha": "3df1a372b844c98c31ec3446faac7c75e64518a3",
"git_sha": "916a4cbc4f831d501860495b157c4857833e22a7",
"installed_by": ["modules"]
},
"tabix/bgziptabix": {
Expand Down
80 changes: 0 additions & 80 deletions modules/nf-core/bcftools/concat/bcftools-concat.diff

This file was deleted.

6 changes: 3 additions & 3 deletions modules/nf-core/bcftools/concat/environment.yml

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

60 changes: 35 additions & 25 deletions modules/nf-core/bcftools/concat/main.nf

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

81 changes: 55 additions & 26 deletions modules/nf-core/bcftools/concat/meta.yml

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

Loading

0 comments on commit 7492949

Please sign in to comment.