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 to parallactic angle calculation #28

Open
henry-ngo opened this issue Jun 5, 2019 · 1 comment
Open

Update to parallactic angle calculation #28

henry-ngo opened this issue Jun 5, 2019 · 1 comment

Comments

@henry-ngo
Copy link
Collaborator

Hello,

I have been attempting some orbit fits using astrometry from VIP-processed NIRC2/vortex images as well as previously published astrometry.

I found that NIRC2_Preprocessing is computing the parallactic angle in a way inconsistent with most of the NIRC2 literature. In particular, there seems to be no correction for the orientation of the NIRC2 detector, as characterized by e.g. Service et al. 2016.

I have implemented an update on the alt_parang branch to add a new keyword that allows use of this distortion solution to compute the parallactic angle. You can see the diff here: master...alt_parang

I would like to determine the origin of the current implementation of the parallactic angle calculation. It would help us determine whether we should replace the calculation with the more commonly used Service+ 2016 solution or provide both choices (and which one should be default). Can someone who wrote the current get_parallactic_angles routine let me know how and why the current calculation is the way it is? @oabsil ? @CARLGOGO ? Thanks!

@oabsil
Copy link
Member

oabsil commented Jun 6, 2019

This is a question for @owertz -- he was the one to implement this. I'm not sure how much testing / validation was done, though.

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

No branches or pull requests

2 participants