Skip to content

Update pixelator to 0.29.0#227

Merged
Aratz merged 21 commits into
devfrom
update-to-pixelator-0.29.0
Jun 22, 2026
Merged

Update pixelator to 0.29.0#227
Aratz merged 21 commits into
devfrom
update-to-pixelator-0.29.0

Conversation

@elhb

@elhb elhb commented Jun 16, 2026

Copy link
Copy Markdown

Summary

Updates the pipeline to pixelator 0.29.0 and pixelatorES 0.11.2, aligning parameters, tests, and snapshots with upstream API and behaviour changes. Includes a coordinated refresh of PNA v2 test fixtures (pool-level denoise, prerelease panel).

Container / dependency updates

  • Bump all local PIXELATOR_* modules from 0.28.0 → 0.29.0
  • Bump EXPERIMENT_SUMMARY (pixelatorES) from 0.10.4 → 0.11.2
  • Add proxiome-v2-immuno-155-prerelease to the PIXELATOR_LIST_OPTIONS stub panel list

Parameter & schema changes

  • Sample calling: change parameter pna_sample_calling_confidence_thresholdpna_sample_calling_enrichment_threshold and wire --enrichment-threshold in conf/modules.pna.config (default is 10.0, pixelator 0.29.0 default)
  • Layout: change production default from wpmds_3dcoarsened_pmds_3d (pixelator 0.29.0 default); add coarsened_pmds_3d to schema enum/default
  • Update nextflow_schema.json defaults and descriptions accordingly

Test profile behaviour

  • test_proxiome_v1.config / test_proxiome_v2.config: override layout to wpmds_3d for CI/minimal tests
    Synthetic nf-core test data still contains graph components too small for coarsened_pmds_3d + default --pmds-pivots 50; production default remains coarsened_pmds_3d
  • Module tests: switch v2 panel references to proxiome-v2-immuno-155-prerelease
  • EXPERIMENT_SUMMARY module test: update denoise fixtures from sample-level (sample1sample4) to pool-level (pool1, pool2) to match v2 pipeline behaviour and pixelatorES 0.11.2 expectations (i.e do denoise before sample calling)
  • PIXELATOR_LAYOUT module test: use pna_layout_layout_algorithm = "coarsened_pmds_3d" (fixes previously ignored layout_algorithm param)

Related PRs

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/pixelator 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).

@elhb elhb requested a review from a team as a code owner June 16, 2026 09:32
@github-actions

Copy link
Copy Markdown

This PR is against the master branch ❌

  • Do not close this PR
  • Click Edit and change the base to dev
  • This CI test will remain failed until you push a new commit

Hi @elhb,

It looks like this pull-request is has been made against the nf-core/pixelator master branch.
The master branch on nf-core repositories should always contain code from the latest release.
Because of this, PRs to master are only allowed if they come from the nf-core/pixelator dev branch.

You do not need to close this PR, you can change the target branch to dev by clicking the "Edit" button at the top of this page.
Note that even after this, the test will continue to show as failing until you push a new commit.

Thanks again for your contribution!

@github-actions

github-actions Bot commented Jun 16, 2026

Copy link
Copy Markdown

nf-core pipelines lint overall result: Passed ✅

Posted for pipeline commit 3400962

+| ✅ 231 tests passed       |+
#| ❔   9 tests were ignored |#
Details

❔ Tests ignored:

  • files_exist - File is ignored: assets/multiqc_config.yml
  • files_exist - File is ignored: conf/igenomes.config
  • files_exist - File is ignored: conf/igenomes_ignored.config
  • files_exist - File is ignored: tests/default.nf.test
  • files_unchanged - File ignored due to lint config: LICENSE or LICENSE.md or LICENCE or LICENCE.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-pixelator_logo_light.png
  • files_unchanged - File ignored due to lint config: .gitignore or .prettierignore
  • multiqc_config - multiqc_config

✅ Tests passed:

Run details

  • nf-core/tools version 4.0.2
  • Run at 2026-06-22 11:46:01

@elhb elhb changed the base branch from master to dev June 16, 2026 09:36
@elhb

elhb commented Jun 17, 2026

Copy link
Copy Markdown
Author

Note: 40dc85f must be reverted before merge to dev!

@Aratz Aratz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great, nice work with the tests!

Comment thread CHANGELOG.md

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looking at the changes I realized the next pipeline version will probably have to be 5.0.0 if we want to stick to semantic versioning 🥲

From https://nf-co.re/docs/specifications/pipelines/requirements/semantic_versioning:

Major Release (e.g., 1.4.3 to 2.0.0)

Changes that are not backwards compatible and may require users to adjust their existing workflows or configurations.

Breaking Changes:

  • ...
  • Renaming or deprecation of existing parameter(s).
  • ...
  • Upgrades to dependencies that introduce breaking changes. For example if the new version of the tool requires additional parameters, removes parameters passed by the pipeline, or changes its outputs in a way that necessitates changes to module output definitions.

@Aratz Aratz merged commit 6375266 into dev Jun 22, 2026
21 checks passed
@elhb elhb deleted the update-to-pixelator-0.29.0 branch June 22, 2026 12:32
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