Skip to content

[13/29] Adopt the selected ZShooter optical prescription and physical slit settings - #15

Merged
Yashvi-Sharma merged 9 commits into
review/xs-12-xshooter-instrumentfrom
review/xs-13-zshooter-prescription
Nov 5, 2026
Merged

Yashvi-Sharma merged 9 commits into
review/xs-12-xshooter-instrumentfrom
review/xs-13-zshooter-prescription

Conversation

@baileyji

@baileyji baileyji commented Oct 23, 2026 •

Copy link
Copy Markdown
Member

Use the selected June 2026 optical prescription, explicit R/H/YJ placements and the agreed omitted reddest YJ order. Configure the physical slit and shared exposure/binning path.

Part 13/29 of the Caltech review series. Review the diff against review/xs-12-xshooter-instrument; the final destination is dev_master.

Merge after series 12: irdb, series 09: scopesim.

Review focus: ZShooter_v2/ZShooter_SPEC.yaml; spectrograph prescription; slits; default.yaml.

Validation: The workspace geometry/sampling tests pass in the 188-case spectroscopy run. Reused slit/exposure and spatial-convergence checks passed in IRDB.

Source commits/snapshots: 0efb498, a31dc90, fc3e2b9, 9fa8317, 2e9a0ae, 72d360a, 75f5f1b, 8c01711.

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.

Resolve configured detector references, supply commands to analytical traces and detector headers, and include configured avalanche photon variance. Use 90,000 samples per region for the sparse dark-only Poisson check.
baileyji and others added 5 commits October 29, 2026 19:46
…zimager-instrument

[28/29] Configure ZImager optics, filters and stochastic detector readouts
…alternate-detectors

[14/29] Configure editable ZShooter detector alternatives and operating points

@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 1e68a6c into review/xs-12-xshooter-instrument 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