Update MIRI detector cross artifact model; fixes #415 #417
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See discussion at #415: the model for the MIRI cross artifact (detector internal diffraction) turned out to be low in webbpsf compared to the MIRI CDP PSFs. I am not sure how this was not caught before. I derived empirical scaling factors that bring them into agreement.
Relatively simple PR in terms of the code; all the effort was in figuring out the appropriate scale factors and verifying things are now working as intended.
This PR assumes that the MIRI CDP PSFs can be taken as a reliable reference for the cross artifact amplitudes vs wavelength.
Before (using original amplitudes):
See difference column at right, then compare to the "after" version of this plot, below..
After (using rescaling factors):
Same plot scale as above.
Horizontal cuts across the PSFs show excellent quantitative agreement now (unlike the prior plots visible in #415):