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

Plot trajectory in ric frame #211

Merged
merged 4 commits into from
Aug 10, 2023

Conversation

ChristopherRabotin
Copy link
Member

Effects

Closes #210

If this is a new feature or a bug fix ...

  • Yes, the branch I'm proposing to merge is called issue-xyz where xyz is the number of the issue.

If this change adds or modifies a validation case

  • No.

But I don't think that's the right approach
Need to refine Python API
+ Plots now use the same color for each data frame
+ RIC error plots now available using the ric_diff_to_parquet function on a spacecraft trajectory
+ Error plots for orbital elements now available using path to parquet or OEM file
@github-actions
Copy link

github-actions bot commented Aug 10, 2023

Visit the preview URL for this PR (updated for commit 4d5268b):

https://nyx-rustdoc--pr211-210-plot-trajectory-dnta0p7v.web.app

(expires Thu, 17 Aug 2023 04:21:28 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: d8e2a55934352d850c15d11866c39eb2d2e029be

Copy link
Member Author

@ChristopherRabotin ChristopherRabotin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@ChristopherRabotin ChristopherRabotin merged commit b6fc505 into master Aug 10, 2023
20 checks passed
@ChristopherRabotin ChristopherRabotin deleted the 210-plot-trajectory-in-ric-frame branch August 10, 2023 05:16
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.

Plot trajectory in RIC frame
1 participant