Skip to content

[12/29] Configure full-frame X-shooter spectroscopy and detector readouts - #14

Merged
Yashvi-Sharma merged 11 commits into
review/xs-11-xshooter-calibrationsfrom
review/xs-12-xshooter-instrument
Nov 5, 2026
Merged

Yashvi-Sharma merged 11 commits into
review/xs-11-xshooter-calibrationsfrom
review/xs-12-xshooter-instrument

Conversation

@baileyji

@baileyji baileyji commented Oct 23, 2026 •

Copy link
Copy Markdown
Member

Configure all three full-frame X-shooter arms, documented slit selections, CCD charge binning, gain/read-noise modes and NIR summed/averaged count conventions.

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

Merge after series 11: irdb, series 10: scopesim.

Review focus: XSHOOTER/default.yaml; XSHOOTER effects/configuration; tests/test_xshooter.py.

Validation: All 36 configured X-shooter cases passed, including full-frame readouts, independent exposure changes, photon rates and mapped line moments.

Source commits/snapshots: 8005be8, 299b3a7, b966cdd, 72d360a, c5766a9, f53d82c, 8c01711, 6cb21a3 (XSHOOTER).

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 6 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
…zshooter-prescription

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

@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 55bc50c into review/xs-11-xshooter-calibrations 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