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/91 position od #115

Merged
merged 3 commits into from
Jul 12, 2023
Merged

Feature/91 position od #115

merged 3 commits into from
Jul 12, 2023

Conversation

thibaudteil
Copy link
Collaborator

  • Tickets addressed: bsk-091
  • Review: By commit
  • Merge strategy: Merge (no squash)

Description

This tickets adds a SRuKF filter to process full camera position vectors. It is nearly an exact replica of the flyby filter, motivating the generation of an interface.

Verification

As with other filters, the test checks for propagation and measurements. The post fit residuals are seen to reconverge into the measurement noise when the dynamics shift.

Documentation

RST documentation

Future work

Refactor part of the interface

@thibaudteil thibaudteil force-pushed the feature/91-positionOD branch 3 times, most recently from 8b24d77 to 3431ff5 Compare July 7, 2023 03:26
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.

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

@thibaudteil thibaudteil force-pushed the feature/91-positionOD branch 3 times, most recently from 0316c19 to cad6822 Compare July 12, 2023 17:59
@thibaudteil thibaudteil merged commit 99856e2 into develop Jul 12, 2023
@thibaudteil thibaudteil deleted the feature/91-positionOD branch July 12, 2023 22:45
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.

2 participants