Skip to content

Add Zimager and XShooter - #11

Open
baileyji wants to merge 100 commits into
dev_masterfrom
zimager_xshooter
Open

baileyji wants to merge 100 commits into
dev_masterfrom
zimager_xshooter

Conversation

@baileyji

Copy link
Copy Markdown
Member

No description provided.

astronomyk and others added 30 commits September 29, 2022 11:59
Update Master for compiliing new MICADO package
Mostly adaption following the off-by-one fix
Merge dev_master in preparation for release
Merge dev_master in preparation for release
Merge dev_master in preparation for release
Using best-guess values for MICADO, for METIS taken from:
AstarVienna#356 (comment)

Closes AstarVienna#356, except the LMS extended mode, which is not yet implemented
anyway (also set the status to concept for that).
The additional diff comes from stripped whitespace or newlines.
These allow the "full detector array" mode to be a "first-order-
configurable", i.e. can be set via `properties`, instead of having to
switch the `.include` on the two effects after instantiating the optical
train. The latter still works, so this is fully backwards compatible.
… configurations**

- Included `sloan_u.dat` and `sloan_z.dat` filters in ZImager optics files.
- Added filter wheel selector with support for `g`, `r`, `i` filters in `ZShooter_IMAGER.yaml`.
- Updated dichroic tree structure with revised `z_order` values.
- Changed selector key for wheels and quantum efficiency curves to `image_plane_id`.
Keep the explicit Zemax grating angles and detector rotations. Record fixed positions in mm: R moves left 150 pixels, H left 130, and YJ up 50 relative to comparison row 2; B, G and K keep their positions. Document the temporary YJ order-57 exclusion without recentering retained orders.

Validation: all 23 retained YJ orders and all 10 H orders fit their full FSR and slit ends; 17 of 20 R orders fit completely.
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.

7 participants