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 NRC coron dispersion coeffs #766

Merged
merged 3 commits into from
Nov 8, 2023
Merged

Conversation

JarronL
Copy link
Collaborator

@JarronL JarronL commented Nov 6, 2023

Update NIRCam coronagraphic wavelength-dependent tilt coefficients. These were miscalculated prior to launch and have since been producing incorrect PSF morphologiers. New values represent actual dispersion from CODEV and Zemax models.

LW example for F444W:
image

SW example for F200W:
image

@JarronL JarronL requested a review from mperrin November 6, 2023 18:04
@mperrin
Copy link
Collaborator

mperrin commented Nov 6, 2023

I confirm this is a necessary fix. I had noticed a while ago the dispersion issue in particular with wide filter coronagraphic PSFs, and brought this to Jarron's attention. Realizing now I had not made a GitHub issue for any of that.

Briefly, for the record:

  • Comparing SW PSFs in F200W, or LW in F356W, F444W showed clearly that the sim speckles did not match well, in particular for the angle or dispersion of the speckles at left and right sides.
    unknown_720
  • Multiplying by an ad-hoc scale factor of -10 resulted in a significantly better fit.
    unknown-5_720
  • For LW I also found discrepant results. Again the sign seems flipped, but the amplitude seems too big instead of too small. Best fit scale factor is like -0.4 for F356W, -0.6 for F444W (very coarsely)

@mperrin
Copy link
Collaborator

mperrin commented Nov 6, 2023

This PR will require updating the test function tests/test_nircam.py/test_nircam_coron_wfe_offset()

@JarronL
Copy link
Collaborator Author

JarronL commented Nov 6, 2023

Done!

Copy link

codecov bot commented Nov 6, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Files Coverage Δ
webbpsf/optics.py 60.66% <100.00%> (ø)

📢 Thoughts on this report? Let us know!

Copy link
Collaborator

@mperrin mperrin 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. Nice improvement in PSFs for wide filters with coronagraphy.

@mperrin
Copy link
Collaborator

mperrin commented Nov 8, 2023

@obi-wan76 FYI this is ready for merging. If you'd like I can go ahead and do so myself, just let me know, but by default I'm assuming you and/or @BradleySappington are handling all the PR merges now.

@BradleySappington BradleySappington merged commit 75824af into develop Nov 8, 2023
@BradleySappington BradleySappington deleted the NRC_coron_dispersion branch November 8, 2023 14:12
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