Skip to content

[28/29] Configure ZImager optics, filters and stochastic detector readouts - #17

Merged
Yashvi-Sharma merged 4 commits into
review/xs-14-alternate-detectorsfrom
review/zi-28-zimager-instrument
Nov 5, 2026
Merged

Yashvi-Sharma merged 4 commits into
review/xs-14-alternate-detectorsfrom
review/zi-28-zimager-instrument

Conversation

@baileyji

@baileyji baileyji commented Oct 23, 2026 •

Copy link
Copy Markdown
Member

Add the configured ZImager optics, dichroics, selectable Sloan filters and three CMOS readouts alongside both spectroscopy modes. Align dark shot noise and exposure warnings with the shared effects.

Part 28/29 of the Caltech review series. Review the diff against review/xs-14-alternate-detectors; the final destination is dev_master.

Merge after series 14: irdb, series 27: scopesim.

Review focus: ZShooter_v2/ZShooter_IMAGER.yaml; ZShooter_IMDET.yaml; filters/optics/dichroics.

Validation: 11 configured imaging/detector-mode/notebook-noise checks passed jointly with series 29. Production files match e332b9e; only the series-14 test-fixture repair differs.

Source commits/snapshots: 3a4cb99, f3e2dd4, 331bfc6, b60c2a5, 69e39b7, 1fca110, c72f9a1, e332b9e.

Use a merge commit to preserve the stack ancestry and pinned commits. After this merges, retarget the next PR in this repository to its default branch. Do not squash/rebase an earlier stack entry without rebuilding its descendants and dependency pins.

@Yashvi-Sharma Yashvi-Sharma left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@Yashvi-Sharma
Yashvi-Sharma merged commit b42f389 into review/xs-14-alternate-detectors Nov 5, 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.

2 participants