Skip to content

Commit

Permalink
Merge pull request #522 from nf-core/fix-null-in-depths
Browse files Browse the repository at this point in the history
Fix null in depth PNG files
  • Loading branch information
jfy133 authored Oct 7, 2023
2 parents 5ea7104 + d511ebd commit 04e2b57
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- [#515](https://github.com/nf-core/mag/pull/515) - Fix overwriting of GUNC output directories when running with domain classification (reported by @maxibor, fix by @jfy133)
- [#516](https://github.com/nf-core/mag/pull/516) - Fix edge-case bug where MEGAHIT re-uses previous work directory on resume and fails (reported by @husensofteng, fix by @prototaxites)
- [#520](https://github.com/nf-core/mag/pull/520) - Fix missing Tiara output files (fix by @jfy133)
- [#522](https://github.com/nf-core/mag/pull/522) - Fix 'nulls' in depth plot PNG files (fix by @jfy133)

### `Dependencies`

Expand Down
4 changes: 2 additions & 2 deletions modules/local/mag_depths_plot.nf
Original file line number Diff line number Diff line change
Expand Up @@ -11,14 +11,14 @@ process MAG_DEPTHS_PLOT {
path(sample_groups)

output:
tuple val(meta), path("${meta.assembler}-${meta.domain}-${meta.binner}-${meta.id}-binDepths.heatmap.png"), emit: heatmap
tuple val(meta), path("${meta.assembler}-${meta.binner}-${meta.id}-binDepths.heatmap.png"), emit: heatmap
path "versions.yml" , emit: versions

script:
"""
plot_mag_depths.py --bin_depths ${depths} \
--groups ${sample_groups} \
--out "${meta.assembler}-${meta.domain}-${meta.binner}-${meta.id}-binDepths.heatmap.png"
--out "${meta.assembler}-${meta.binner}-${meta.id}-binDepths.heatmap.png"
cat <<-END_VERSIONS > versions.yml
"${task.process}":
Expand Down

0 comments on commit 04e2b57

Please sign in to comment.