Skip to content

fix_illum: pool tiles across Z + darkfield/fit knobs (chain tip = dev)#125

Open
FIrgolitsch wants to merge 2 commits into
pr-v-psf-focalfrom
pr-u-fix-illum
Open

fix_illum: pool tiles across Z + darkfield/fit knobs (chain tip = dev)#125
FIrgolitsch wants to merge 2 commits into
pr-v-psf-focalfrom
pr-u-fix-illum

Conversation

@FIrgolitsch

@FIrgolitsch FIrgolitsch commented May 20, 2026

Copy link
Copy Markdown
Contributor

Stacked PR 22/22 — review order: #115#97#98#99#100#101#108#106#107#87#116#110#111#40#112#113#117#118#120#121#122#123#124#125

Base: pr-v-psf-focal. Retargets to main as upstream PRs merge. Final PR — after this merges, main matches dev.


fix_illum: pool tiles across Z + darkfield/fit knobs

Stacked on pr-v-psf-focal. Tip of the chain — equals dev.

Commits

  • feat(fix_illum): pool tiles across Z, expose darkfield/fit_max_samples/max_iterations (squashed from 5 commits) — linum_compensate_illumination.py now pools tiles across Z slices for a more stable flat-field estimate and exposes three new knobs in nextflow.config:
    • fix_illum_darkfield (bool) — fit a darkfield term
    • fix_illum_fit_max_samples (int) — cap samples per fit for speed
    • fix_illum_max_iterations (int) — BaSiC solver iteration cap
  • chore: update uv.lock (squashed)

Docs gaps

The three new fix_illum_* params are not yet in docs/NEXTFLOW_WORKFLOWS.md; follow-up doc PR.

Chain summary

This is PR 22/22. Merging the full chain into main reproduces dev exactly (verified tree-identical).

…s/max_iterations parameters

Squashed from:
  92961af perf(fix_illum): pool tiles across Z and enable darkfield for stable flatfield
  421281b Revert darkfield defaulting to true
  6290675 expose fix_illum_fit_max_samples and fix_illum_max_iterations pipeline params
  37f55ce expose fix_illum_darkfield pipeline param to remove residual per-tile waffle
  7b0551d Fix darkfield estimation and filtering for illumination correction
Squashed from:
  77a09d6 Updated uv lock file
  ff1ff79 Updated uv lock file
This was referenced May 20, 2026
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.

1 participant