-
Notifications
You must be signed in to change notification settings - Fork 37
Add Subaru and SCExAO apertures #243
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
From in-person conversation with Emiel
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #243 +/- ##
==========================================
+ Coverage 84.82% 84.93% +0.10%
==========================================
Files 104 104
Lines 8008 8057 +49
==========================================
+ Hits 6793 6843 +50
+ Misses 1215 1214 -1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
@mileslucas Only waiting on a picture of a CAD drawing of the pupil and the HCIPy pupil fits file in DS9 next to it. Otherwise this is fully ready to go. If you're not allowed to share that, or if that's too much to ask, or if that's difficult (read: >5mins of work) to look up, then I'm fine with merging without that confirmation. |
|
Thanks, we got some drawings made for the pupils--for now just on Google Drive. I'll get the references checked and try and close this out next week. |
|
Okay, here is a reference for the SCExAO pupil mask: https://drive.google.com/file/d/1WddQr1quYshpPvvIbGt3hRzdmHnb4gT1/view?usp=drive_link Here is a matplotlib figure generated via overlaid with 50% opacity on the mask CAD drawing. |
|
Thanks Emiel! I dragged my feet a bit and I appreciate you carrying this over the finish line. |
Adds functions for generating Subaru and SCExAO pupils. The numbers are based on private communication with Julien Lozi, which are published in a table in Lucas et al., 2024. The code is based on a Lyot stop generator I made a while ago, which allows arbitrarily scaling inner and outer diameters, as well as spider widths. The difference between the Subaru geometry and SCExAO geometry is that SCExAO has a very slightly undersized pupil compared to Subaru, plus the SCExAO aperture has masks for bad actuators.