Skip to content

Receivers take the components that split windows and layers (0.4.0) - #33

Merged
trissim merged 5 commits into
mainfrom
refactor/G6-viewer-slot-families
Oct 10, 2026
Merged

trissim merged 5 commits into
mainfrom
refactor/G6-viewer-slot-families

Conversation

@trissim

@trissim trissim commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

Lockstep with OpenHCS G6 (domain-blind viewers) and ZMQRuntime #27.

  • group_items_into_windows(items, display_layout, window_components) replaces group_items_by_component_modes: the caller (the OpenHCS Fiji viewer, from its own slot family) passes the components that get a window each. GroupedWindowItems drops the channel/slice/frame lists the helper only passed through.
  • build_route_key(..., layer_components, ...) takes the napari layer components instead of reading the flat ViewerComponentMode.
  • Deleted: normalize_component_layout (test-only), NapariDisplayWireField, FijiDisplayWireField (only OpenHCS used them; the OpenHCS display-settings dataclass fields are the wire keys now).

Version bumped to 0.4.0 (breaking) with zmqruntime>=0.5.0,<0.6; not tagged. Do not merge before the OpenHCS G6 PR.

Tests: full PolyStore suite 588 passed, 2 skipped.

🤖 Generated with Claude Code

…0.3.5

The Fiji window grouping and the napari route key re-read zmqruntime's flat
ViewerComponentMode enum. The calling viewer now passes the components its
own slot family places in separate windows or layers, so these helpers name
no viewer vocabulary. normalize_component_layout (test-only) and the
NapariDisplayWireField/FijiDisplayWireField rosters (the OpenHCS display
settings dataclasses are the wire keys now) are deleted.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 10, 2026 07:55

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

The receiver API change is breaking, and zmqruntime 0.5 deletes the mode enum
these receivers used to read.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@trissim trissim changed the title Receivers take the components that split windows and layers (0.3.5) Receivers take the components that split windows and layers (0.4.0) Oct 10, 2026
trissim and others added 3 commits October 10, 2026 12:01
@trissim
trissim merged commit f6af43a into main Oct 10, 2026
11 checks passed
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