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

Consider adding v2a_ref, v3a_ref to wcsinfo #437

Open
schlafly opened this issue Aug 27, 2024 · 0 comments
Open

Consider adding v2a_ref, v3a_ref to wcsinfo #437

schlafly opened this issue Aug 27, 2024 · 0 comments

Comments

@schlafly
Copy link
Collaborator

Conceptually I had associated v2_ref, v3_ref with the v2/v3 location corresponding to radec_ref. That's what the fields currently mean in Roman.

However, in Webb, the aberration correction is applied after v2/v3 ref, so that instead an "aberrated v2/v3" reference position is what corresponds to ra/dec_ref. This is close to the original v2/v3 ref but scaled by va_scale.

We should consider whether we want to also provide these aberrated v2/v3 reference positions, which would make it so that we are providing the v2/v3 reference positions that actually correspond to ra/dec reference points. Or alternatively add new ra/dec_ref that correspond to the post-aberration locations of v2/v3_ref.

@schlafly schlafly transferred this issue from spacetelescope/roman_datamodels Aug 27, 2024
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

1 participant