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

Lunar reconnaisance orbiter example #356

Merged
merged 20 commits into from
Aug 19, 2024
Merged

Conversation

ChristopherRabotin
Copy link
Member

@ChristopherRabotin ChristopherRabotin commented Aug 17, 2024

Summary

Adds an example where the orbital dynamics of the LRO spacecraft are nearly matched, and then an orbit determination simulation and estimation is conducted on that orbit.

Closes #312.

Architectural Changes

No change

New Features

  • Add building a Nyx trajectory from a BSP file
  • Account for eclipsing bodies in tracking data computation (this will eventually be part of ANISE).

Improvements

No change

Bug Fixes

  • Fix computation of the residual ratios

Testing and validation

Verification of the implementation of the GRAIL JGGRX model in the Moon Principal Axes frame

Documentation

The LRO case is thoroughly documented.

Copy link

github-actions bot commented Aug 17, 2024

Visit the preview URL for this PR (updated for commit 0f1a09c):

https://nyx-rustdoc--pr356-312-lro-estimation-f6jgdwu1.web.app

(expires Mon, 26 Aug 2024 06:15:24 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: d8e2a55934352d850c15d11866c39eb2d2e029be

@ChristopherRabotin
Copy link
Member Author

To-do: investigate why the measurement noise values are so low when plotting the residuals.

Also dive into the ODTK math spec again for the measurement ratios to see in which cases these get rejected. I think my logic matches that of odtk, but I should double check that. In my tests tonight it seems that the filter is overzealous in rejecting measurements.

- Fix computation of residual ratios
- Renamed msr_noise to msr_covar
- Fix storing of the tracker msr noise
- Fix computation of the variance of StochasticNoise
@ChristopherRabotin ChristopherRabotin merged commit 050cd00 into master Aug 19, 2024
6 checks passed
@ChristopherRabotin ChristopherRabotin deleted the 312-lro-estimation branch August 19, 2024 06:34
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

Successfully merging this pull request may close these issues.

Nyx shall switch to the principal axes frame of the Moon for harmonics computations
1 participant