-
Notifications
You must be signed in to change notification settings - Fork 62
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
improve MIRI LRS model details #787
Conversation
0ca6b20
to
657d70a
Compare
Hello @mperrin, Thank you for updating !
Comment last updated at 2024-04-29 21:28:55 UTC |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #787 +/- ##
===========================================
+ Coverage 59.20% 59.54% +0.33%
===========================================
Files 16 16
Lines 6955 6926 -29
===========================================
+ Hits 4118 4124 +6
+ Misses 2837 2802 -35 ☔ View full report in Codecov by Sentry. |
ad1539e
to
b7b4a39
Compare
b7b4a39
to
8afebc7
Compare
In the course of some science analyses and discussions, I discovered/realized that the LRS calc has been (until now) using a very coarse pixel scale via default FFT propagation from pupil to image plane. This is not sufficiently precise. Much better outputs achieved via using an adjustable matrix Fourier transform to set much finer spatial scale right around the slit. As a bonus, this actually makes the code faster too since in this case we can restrict to computing a quite small region of the image plane. But that's not the main motivation. |
… WFE to be after coord rotation for LRS (also fixes for coronagraphy incidentally)
…including imaging filters
…t silently ignore/skip SIAF distortion for MIRI LRS; add basic unit test for LRS
…much finer pixelscale via MFT
25307d8
to
b635325
Compare
@mperrin how do I get the same image/output that you are showing in your comment above? (with the LRS slit). I checked your modifications and I was able to create a LRS PSF based on the test setup but I would like to play around a bit more, specially to get the same images. Thanks! |
The above plots were obtained by using the
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me. I did the example from the test, the intermediate planes, and double check the values from the papers.
Address #785.
Remaining open items from #785:
Also:
Defer these as out-of-scope; could be done in some future PR: