Skip to content

Add a 4096-pixel H4RG K detector mode - #18

Merged
Yashvi-Sharma merged 2 commits into
review/zi-28-zimager-instrumentfrom
feat/h4rg-k
Nov 5, 2026
Merged

Yashvi-Sharma merged 2 commits into
review/zi-28-zimager-instrumentfrom
feat/h4rg-k

Conversation

@baileyji

Copy link
Copy Markdown
Member

Add SPEC_ALTDETECT4K, retaining the alternate package in BLUE/GREEN/RED/YJ/H and replacing K with a 4096 by 4096 H4RG at 10 micron pitch. Detector dimensions and analytical-trace bounds now use the same configured K dimensions; the existing SPEC and H2RG modes explicitly retain 2048 by 2048 K detectors and the physical optical prescription is unchanged.

The H4RG package reports SPEC_ALTDETECT4K in detector headers. This corrects the initial local configuration that produced 4096-pixel H4RG images labeled SPEC_ALTDETECT.

Based on PR #17. Reductions of correctly labeled H4RG frames require zshooter-drp #9. Use a merge commit and retarget to dev_master after #17 merges.

Validation: three configured OpticalTrain cases passed for SPEC, H2RG and H4RG, checking mode identity, detector type, dimensions, physical pixel pitch and all eight modeled K orders. The H4RG metadata fix changes neither detector geometry nor noise parameters.

@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 7f0e802 into review/zi-28-zimager-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