-
Notifications
You must be signed in to change notification settings - Fork 1
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
MAINT: Meta header for propagators #122
Conversation
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.
Style suggestion for legibility.
Co-authored-by: Angeline Burrell <[email protected]>
@jklenzing I approved, but just noticed that 3.10 is failing. :( Up to you on solving that. |
It's related to apexpy. Will probably just turn that bit off. |
apexpy is only installed for the older numpy. This was implemented in the previous version due to poor compatibility with numpy 2.0.0. Since this is optional, other workflows skip over apexpy-related tests. Bumping our minimum numpy to match SPEC 0 (1.23 -> 1.24) seems to fix the issue for now. |
Description
Adds orbital info used for propagation to global metadata values. If Keplerian elements are used in generation, then only those elements (including default values) are added. If TLEs are used, only those default values are used.
Type of change
How Has This Been Tested?
Loading an instrument and inspecting for header data.
Checklist:
develop
(notmain
) branchCHANGELOG.md
, summarizing the changesIf this is a release PR, replace the first item of the above checklist with the
release checklist on the pysat wiki:
https://github.com/pysat/pysat/wiki/Checklist-for-Release