-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add PA correction for NIRISS #139
Conversation
for more information, see https://pre-commit.ci
Set to None regardless of the reason
Could be that instrument is set to NIRISS but the data was simulated or modified. This will return 0.0 by default.
For both *parang functions and for the compute_pa() function
for more information, see https://pre-commit.ci
I added tests for SPHERE and AMI PA calculations. I think this is ready for review @DrSoulain. |
Had a quick look and I would also like to review this from a technical angle. Please ping me if I forget ! |
I'll probably just approve this next time, but I'd like to see #136 merged first |
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.
Thanks @vandalt for the implementation of the PA computation for NIRISS. The PA (orientation of the mask compared to the north-up, east-left) is now automaticaly saved with AMICAL.
I modified the function used for SPHERE to implement options for different instruments and added a function to compute it for NIRISS. This depends on #136, so marking as Draft for now.