Skip to content

refactor: optimize process labels, tune slurm executor settings, adju…#73

Merged
JTL-lab merged 1 commit into
mainfrom
refactor/resource-requirements
Mar 21, 2026
Merged

refactor: optimize process labels, tune slurm executor settings, adju…#73
JTL-lab merged 1 commit into
mainfrom
refactor/resource-requirements

Conversation

@JTL-lab

@JTL-lab JTL-lab commented Mar 21, 2026

Copy link
Copy Markdown
Collaborator

…st default params

refactor: update cpus, memory, and time in base.config process label definitions
refactor: reassign process labels in module main.nf to better match actual resource needs
refactor: tune SLURM executor settings in slurm.config (pollInterval, queueStatInterval,  submitRateLimit, exitReadTimeout)
refactor: adjust default params in nextflow.config
refactor: add process-specific time limits in modules.config (e.g. MSCONVERT_MZML)
fix: hardcoded 'merged' meta.id in pacbio_isocall.nf to use params.dataset_name (edited)
fix: removes spaces from BED12 name field in novel_peptide.R

PR checklist

  • 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
  • Make sure your code lints (nf-core pipelines lint).
  • Ensure the test suite passes (nextflow run . -profile test,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).

…st default params

refactor: update cpus, memory, and time in base.config process label definitions
refactor: reassign process labels in module main.nf to better match actual resource needs
refactor: tune SLURM executor settings in slurm.config (pollInterval, queueStatInterval,  submitRateLimit, exitReadTimeout)
refactor: adjust default params in nextflow.config
refactor: add process-specific time limits in modules.config (e.g. MSCONVERT_MZML)
fix: hardcoded 'merged' meta.id in pacbio_isocall.nf to use params.dataset_name (edited)
fix: removes spaces from BED12 name field in novel_peptide.R
@JTL-lab JTL-lab merged commit dd882d0 into main Mar 21, 2026
12 of 25 checks passed
@JTL-lab JTL-lab deleted the refactor/resource-requirements branch May 16, 2026 14:23
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.

2 participants