Skip to content
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

Update MIRI detector cross artifact model; fixes #415 #417

Merged

Conversation

mperrin
Copy link
Collaborator

@mperrin mperrin commented Feb 24, 2021

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..

Unknown-4

After (using rescaling factors):

Same plot scale as above.

Unknown-7

Horizontal cuts across the PSFs show excellent quantitative agreement now (unlike the prior plots visible in #415):

Unknown-9

@mperrin
Copy link
Collaborator Author

mperrin commented Feb 24, 2021

@merope82 would you be so kind as to take a look at the plots on this update to WebbPSF, please? It's pretty quick and dirty and ad hoc, but I think suffices for current purposes.

@codecov
Copy link

codecov bot commented Feb 24, 2021

Codecov Report

Merging #417 (b642091) into develop (67c7c18) will increase coverage by 0.02%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #417      +/-   ##
===========================================
+ Coverage    48.59%   48.62%   +0.02%     
===========================================
  Files           14       14              
  Lines         5772     5775       +3     
===========================================
+ Hits          2805     2808       +3     
  Misses        2967     2967              
Impacted Files Coverage Δ
webbpsf/distortion.py 95.23% <100.00%> (+0.11%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 67c7c18...b642091. Read the comment docs.

@merope82
Copy link

merope82 commented Feb 24, 2021 via email

@mperrin mperrin requested a review from shanosborne February 26, 2021 19:26
Copy link
Contributor

@shanosborne shanosborne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! I'm glad to see we got some more accurate numbers

@mperrin mperrin merged commit b4b1e05 into spacetelescope:develop Mar 1, 2021
@mperrin mperrin deleted the rescale_miri_cross_artifact_model branch March 1, 2021 19:35
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.

3 participants