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

Feature/703 ephemeris based guidance #245

Merged
merged 5 commits into from
May 24, 2024

Conversation

JulianHammerl
Copy link
Collaborator

  • Tickets addressed: MAXGNC-703
  • Review: By commit
  • Merge strategy: Merge (no squash)

Description

A module was created that takes the ephemeris message containing the position and velocity of two objects (a base object and a secondary object), and computes the relative position and velocity of the secondary with respect to the base. This is then written to a navigation and filter message.

Verification

A UnitTest was added.

Documentation

An rst documentation was added.

Future work

N/A

@JulianHammerl JulianHammerl added the enhancement New feature or request label May 15, 2024
@JulianHammerl JulianHammerl self-assigned this May 15, 2024
@JulianHammerl JulianHammerl force-pushed the feature/703-ephemeris-based-guidance branch from 678e133 to 5b97cf1 Compare May 22, 2024 15:25
Copy link
Collaborator

@patkenneally patkenneally left a comment

Choose a reason for hiding this comment

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

🔻 Issues to address before merge
🔶 Requests that should not block merge, but should at least be discussed
🔵 Recommendations that can be ignored if desired

Nicely done.

@JulianHammerl JulianHammerl force-pushed the feature/703-ephemeris-based-guidance branch from 5b97cf1 to 4797b5a Compare May 23, 2024 22:12
@JulianHammerl JulianHammerl force-pushed the feature/703-ephemeris-based-guidance branch from 4797b5a to 9bc6dee Compare May 24, 2024 17:22
@patkenneally patkenneally merged commit fb7865c into develop May 24, 2024
3 checks passed
@patkenneally patkenneally deleted the feature/703-ephemeris-based-guidance branch May 24, 2024 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants