Skip to content

Conversation

@quentinblampey
Copy link
Collaborator

@quentinblampey quentinblampey commented Jul 24, 2025

New release PR after recent updates (moving processes to modules, and using -params-file instead of --configfile).

The conda tests are broken for two profiles because some processes require the installation of a specific command-line tool. It works well using docker or singularity though. Do you know how I could fix this? Or maybe prevent using conda for some specific profiles (e.g., baysor and proseg).

EDIT: actually, there is also an issue with singularity on test_cellpose. This is due to a permission issue to write in the HOME directory. The lines below fixed the issue for Docker, but apparently not for Singularity...

withName: PATCH_SEGMENTATION_CELLPOSE {
    containerOptions = ''
}

Happy to hear from your review @mashehu @kenibrewer!

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
  • If necessary, also make a PR on the nf-core/sopa 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>).
  • 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).

quentinblampey and others added 30 commits October 14, 2025 09:51
Important! Template update for nf-core/tools v3.4.1
Update to new runner size syntax
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.

5 participants