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.

Sign up for GitHub

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

Jump to bottom

Allow extrapolation for more than 10 nanoseconds #346

Closed
ChristopherRabotin opened this issue Nov 4, 2024 · 0 comments · Fixed by #347
Closed

Allow extrapolation for more than 10 nanoseconds #346

ChristopherRabotin opened this issue Nov 4, 2024 · 0 comments · Fixed by #347
Milestone
Version 0.5.0: Py...

Comments

@ChristopherRabotin
Copy link
Member

Bug report

Describe the bug

When converting from ET/TDB to UTC and back, some rounding may happen. In one specific example, there were 25 nanoseconds of differences. Querying an ephemeris at the edge of its domain would then error as a direct implication of the accumulation of this rounding error.

Expected behavior

I expect to be able to query my ephemeris.
@ChristopherRabotin ChristopherRabotin mentioned this issue Nov 4, 2024
Merged
ChristopherRabotin added a commit that referenced this issue Nov 4, 2024
@ChristopherRabotin
Merge pull request #347 from nyx-space/gh-346
2086cc6 
Allow up to 100 ns of leeway on Hermite and Lagrange querying
ChristopherRabotin added a commit that referenced this issue Nov 5, 2024
@ChristopherRabotin
Fix #346 and add relevant test
edcc718 
Increased the acceptance limit for SPICE validation on LRO because it seems to depend on the platform
ChristopherRabotin added a commit that referenced this issue Nov 5, 2024
@ChristopherRabotin
Merge pull request #348 from nyx-space/gh-346-bis
e5b8d3e 
Allow 100 nanosecond leeway in querying DAF files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Version 0.5.0: Python type hint + dep...
Development

Successfully merging a pull request may close this issue.

Allow up to 100 ns of leeway on Hermite and Lagrange querying nyx-space/anise
1 participant
@ChristopherRabotin