Skip to content

feat(cytoland): non-square rotation TTA + reuse helpers#465

Merged
ieivanov merged 1 commit into
mainfrom
feat/vs-nonsquare-tta-and-helpers-main
Jun 10, 2026
Merged

feat(cytoland): non-square rotation TTA + reuse helpers#465
ieivanov merged 1 commit into
mainfrom
feat/vs-nonsquare-tta-and-helpers-main

Conversation

@edyoshikun

Copy link
Copy Markdown
Member

Ports the changes from #450 onto main.

#450 was accidentally squash-merged into modular-viscy-staging (commit 674dbeef) instead of main. Because the modular-viscy-staging → main integration (#373) had already merged on 2026-06-04 — six days before #450 — the change never reached main and won't arrive via any future staging merge (staging is now retired). This PR cherry-picks the squashed commit onto main (clean, zero conflicts — the five touched files were byte-identical between the two branches).

Original change (unchanged):

  • Fix AugmentedPredictionVSUNet._predict_with_tta to crop to the augmented (post-transform) spatial shape, so shape-changing transforms (90/270° rotations that swap Y/X) work on non-square FOVs.
  • Add rotation_tta_transforms() factory and AugmentedPredictionVSUNet.with_rotation_tta() convenience constructor.
  • Make _read_norm_meta canonical in viscy_data._utils and export public alias read_norm_meta.
  • Regression tests for non-square rotation TTA and the helper.

Supersedes #450 for the main line. The copy on modular-viscy-staging is left as-is (branch retired).

🤖 Generated with Claude Code

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
(cherry picked from commit 674dbee)
@ieivanov ieivanov merged commit 7fb3bc1 into main Jun 10, 2026
28 checks passed
@ieivanov ieivanov deleted the feat/vs-nonsquare-tta-and-helpers-main branch June 10, 2026 22:18
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