Skip to content

Migrate file publishing from publishDir to centralized output {} block - I #784

Merged
ramprasadn merged 25 commits into
devfrom
outputblock
Mar 17, 2026
Merged

Migrate file publishing from publishDir to centralized output {} block - I #784
ramprasadn merged 25 commits into
devfrom
outputblock

Conversation

@ramprasadn
Copy link
Copy Markdown
Collaborator

PR checklist

Replaces per-process publishDir config directives with a centralized publishing pattern. Each subworkflow now builds a ch_publish channel emitting [destination, value] tuples that bubble up to main.nf, where a single publish: + output {} block handles all file publishing.

The following published files are affected by this change (all verified to be present in the same location after the change)

results_subwfs/call_mobile_elements/justhusky_mobile_elements.vcf.gz
results_subwfs/call_mobile_elements/justhusky_mobile_elements.vcf.gz.tbi

results_subwfs/call_snv/genome/justhusky_snv.vcf.gz
results_subwfs/call_snv/genome/justhusky_snv.vcf.gz.tbi

results_subwfs/call_snv/mitochondria/justhusky_mitochondria.vcf.gz
results_subwfs/call_snv/mitochondria/justhusky_mitochondria.vcf.gz.tbi

results_subwfs/call_sv/genome/justhusky_sv.vcf.gz
results_subwfs/call_sv/genome/justhusky_sv.vcf.gz.tbi

results_subwfs/call_sv/mitochondria/earlycasualcaiman_LNUMBER1.breakpoint
results_subwfs/call_sv/mitochondria/earlycasualcaiman_mitochondria_deletions.txt
results_subwfs/call_sv/mitochondria/earlycasualcaiman_LNUMBER1.cluster

results_subwfs/call_sv/mitochondria/slowlycivilbuck_LNUMBER2.breakpoint
results_subwfs/call_sv/mitochondria/slowlycivilbuck_mitochondria_deletions.txt
results_subwfs/call_sv/mitochondria/hugelymodelbat_LNUMBER3.cluster

results_subwfs/call_sv/mitochondria/hugelymodelbat_LNUMBER3.breakpoint
results_subwfs/call_sv/mitochondria/hugelymodelbat_mitochondria_deletions.txt
results_subwfs/call_sv/mitochondria/slowlycivilbuck_LNUMBER2.cluster

results/alignment/earlycasualcaiman_mt_subsample.bam
results/alignment/earlycasualcaiman_mt_subsample.bam.bai
results/alignment/earlycasualcaiman_sorted_md.metrics.txt
results/alignment/earlycasualcaiman_sorted_md.bam
results/alignment/earlycasualcaiman_sorted_md.bam.bai

results/alignment/hugelymodelbat_mt_subsample.bam
results/alignment/hugelymodelbat_mt_subsample.bam.bai
results/alignment/hugelymodelbat_sorted_md.metrics.txt
results/alignment/hugelymodelbat_sorted_md.bam
results/alignment/hugelymodelbat_sorted_md.bam.bai

results/alignment/slowlycivilbuck_mt_subsample.bam
results/alignment/slowlycivilbuck_mt_subsample.bam.bai
results/alignment/slowlycivilbuck_sorted_md.metrics.txt
results/alignment/slowlycivilbuck_sorted_md.bam
results/alignment/slowlycivilbuck_sorted_md.bam.bai

And 159 files in qc_bam folder 😅

  • This comment contains a description of changes (with reason).
  • If you've fixed a bug or added code that should be tested, add tests!
  • If you've added a new tool - have you followed the pipeline conventions in the contribution docs
  • If necessary, also make a PR on the nf-core/raredisease branch on the nf-core/test-datasets repository.
  • Make sure your code lints (nf-core pipelines lint).
  • Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
  • Ensure the test suite passes (nextflow run . -profile test_singleton,docker --outdir <OUTDIR>).
  • Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
  • Usage Documentation in docs/usage.md is updated.
  • Output Documentation in docs/output.md is updated.
  • CHANGELOG.md is updated.
  • README.md is updated (including new tool citations and authors/contributors).

@nf-core-bot
Copy link
Copy Markdown
Member

Warning

Newer version of the nf-core template is available.

Your pipeline is using an old version of the nf-core template: 3.5.1.
Please update your pipeline to the latest version.

For more documentation on how to update your pipeline, please see the nf-core documentation and Synchronisation documentation.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 12, 2026

nf-core pipelines lint overall result: Passed ✅ ⚠️

Posted for pipeline commit 57fa0af

+| ✅ 234 tests passed       |+
#| ❔   7 tests were ignored |#
#| ❔   1 tests had warnings |#
!| ❗   8 tests had warnings |!
Details

❗ Test warnings:

  • pipeline_if_empty_null - ifEmpty(null) found in /home/runner/work/raredisease/raredisease/subworkflows/local/align_sentieon/main.nf: _ mq_metrics = SENTIEON_DATAMETRICS.out.mq_metrics.ifEmpty(null) // channel: [ val(meta), path(mq_metrics) ]
    _
  • pipeline_if_empty_null - ifEmpty(null) found in /home/runner/work/raredisease/raredisease/subworkflows/local/align_sentieon/main.nf: _ qd_metrics = SENTIEON_DATAMETRICS.out.qd_metrics.ifEmpty(null) // channel: [ val(meta), path(qd_metrics) ]
    _
  • pipeline_if_empty_null - ifEmpty(null) found in /home/runner/work/raredisease/raredisease/subworkflows/local/align_sentieon/main.nf: _ gc_metrics = SENTIEON_DATAMETRICS.out.gc_metrics.ifEmpty(null) // channel: [ val(meta), path(gc_metrics) ]
    _
  • pipeline_if_empty_null - ifEmpty(null) found in /home/runner/work/raredisease/raredisease/subworkflows/local/align_sentieon/main.nf: _ gc_summary = SENTIEON_DATAMETRICS.out.gc_summary.ifEmpty(null) // channel: [ val(meta), path(gc_summary) ]
    _
  • pipeline_if_empty_null - ifEmpty(null) found in /home/runner/work/raredisease/raredisease/subworkflows/local/align_sentieon/main.nf: _ aln_metrics = SENTIEON_DATAMETRICS.out.aln_metrics.ifEmpty(null) // channel: [ val(meta), path(aln_metrics) ]
    _
  • pipeline_if_empty_null - ifEmpty(null) found in /home/runner/work/raredisease/raredisease/subworkflows/local/align_sentieon/main.nf: _ is_metrics = SENTIEON_DATAMETRICS.out.is_metrics.ifEmpty(null) // channel: [ val(meta), path(is_metrics) ]
    _
  • schema_lint - Input mimetype is missing or empty
  • schema_description - No description provided in schema for parameter: hisat2

❔ Tests ignored:

  • files_exist - File is ignored: conf/modules.config
  • files_unchanged - File ignored due to lint config: .github/CONTRIBUTING.md
  • files_unchanged - File ignored due to lint config: .github/PULL_REQUEST_TEMPLATE.md
  • files_unchanged - File ignored due to lint config: assets/nf-core-raredisease_logo_light.png
  • files_unchanged - File ignored due to lint config: docs/images/nf-core-raredisease_logo_light.png
  • files_unchanged - File ignored due to lint config: docs/images/nf-core-raredisease_logo_dark.png
  • modules_config - modules_config

❔ Tests fixed:

✅ Tests passed:

Run details

  • nf-core/tools version 3.5.1
  • Run at 2026-03-16 18:06:46

@ramprasadn ramprasadn marked this pull request as ready for review March 13, 2026 23:32
Comment thread nextflow.config
@ramprasadn ramprasadn requested a review from maxulysse March 16, 2026 19:29
Copy link
Copy Markdown
Member

@maxulysse maxulysse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good to me, only changes in the snapshots are in the local subworkflows

@ramprasadn ramprasadn merged commit 875a6f0 into dev Mar 17, 2026
125 of 131 checks passed
@ramprasadn ramprasadn deleted the outputblock branch March 30, 2026 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants