-
Notifications
You must be signed in to change notification settings - Fork 63
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
WFR4+MIMF_KDP: add LookupTable optic class for Ball-delivered OPDs #328
Conversation
Pull Request Test Coverage Report for Build 267
💛 - Coveralls |
8d7b09b
to
64b4295
Compare
Rebased onto current develop. |
Codecov Report
@@ Coverage Diff @@
## develop #328 +/- ##
===========================================
- Coverage 64.06% 62.58% -1.48%
===========================================
Files 11 11
Lines 4352 4456 +104
===========================================
+ Hits 2788 2789 +1
- Misses 1564 1667 +103
Continue to review full report at Codecov.
|
496777c
to
6901f52
Compare
6901f52
to
4659e00
Compare
This is old code, developed specifically for WFR4 as a short-term implementation, and is not nearly as rigorous as the subsequent implementations of better ways to exchange information between ITM and WebbPSF+MIRAGE. I don't see any need to keep or merge this branch so I'm going to close this PR. |
Looks like we may need this again after all for the MIMF KDP simulation dataset... |
4659e00
to
57e3af4
Compare
Updated for 2021 March MIMF-2 Key Decision Point practice. See also notebooks in |
…joie_import_ball_phasemaps
For Ball LookupTableOPD, add support for general defocus, e.g. for MIMF
This functionality has been useful in enough team practices and LREs and may well be useful in the future at some point. Rather than keeping this PR open indefinitely I'm going to go ahead and merge. This is an optional class, the use of which requires manually selecting it in to replace the default (as is done in the LRE and MIMF KDP notebooks). Merging it will have no impact on the results of other calculations with webbpsf. |
… merged in spacetelescope#328 but weren't, by accident
… merged in spacetelescope#328 but weren't, by accident
Add a new optic class that reads OPDs from a lookup table provided by Ball, intended specifically for Wavefront Rehearsal 4 (WFR4) sim data generation in Nov/Dec 2019.
This is not general-purpose, and more of a hack than usual in webbpsf... Not sure if I will actually merge this or not; we can just use this branch for the WFR4 sims.
See also the notebooks in
/ifs/jwst/tel/wfr4_mirage_sims/phase_maps_from_ball/
FYI @obi-wan76 @Skyhawk172 @shanosborne