-
Notifications
You must be signed in to change notification settings - Fork 40
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 yaml_generator to properly handle non-sidereal sources #590
Update yaml_generator to properly handle non-sidereal sources #590
Conversation
Tests show that this seems to be working well. It has revealed a bug in the pointing values that are calculated using ra_dec_update(), but the bug appears to affect all modes (not just non-sidereal) and is not caused by these changes, but instead is already present in the master branch. I'll address that in a separate issue and PR. |
Nevermind about the ra_dec_update bug. Turns out APT was using a different version of the PRD compared to what pysiaf was using. This was causing sources to be shifted from their expected locations. |
This PR allows the yaml generator to calculate and use the correct pointings for non-sidereal observations, based on an ephemeris file or manually input velocity values in the non-sidereal source catalog.
Note that I made a bunch of updates to apt_inputs.py as well, and then accidentally did
git checkout
instead ofgit add
on the file, so I lost those changes and need to redo them.Resolves #588