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

New state estimation #239

Merged
merged 33 commits into from
Sep 23, 2023
Merged

New state estimation #239

merged 33 commits into from
Sep 23, 2023

Conversation

antoinedang
Copy link
Contributor

Related Issue(s)

Other Dependent PRs

Changes Made

  • Describe the changes made in this PR.

Rationale/Other Solutions Considered

  • Explain the rationale behind this PR.
  • Mention any alternative solutions considered (if any).

Tests Done

  • Describe the tests that have been performed.
  • Mention any relevant test cases or scenarios.

Optional Screen Recording

Attach a screen recording of test run(s) (if applicable).

Required Wiki Changes (if any)

  • List any changes that need to be made to the project's wiki (if applicable).

Required install_dependencies.sh Changes (if any)

  • List any new dependencies that need to be installed by the install_dependencies.sh script

Related Branches (other than one that will be merged)

  • List any related branches (if applicable).

@MrMondrian MrMondrian closed this Sep 9, 2023
@MrMondrian MrMondrian reopened this Sep 9, 2023
@MrMondrian MrMondrian closed this Sep 9, 2023
@antoinedang antoinedang reopened this Sep 19, 2023
@MrMondrian
Copy link
Contributor

This works and is ready for rigorous testing

@MrMondrian
Copy link
Contributor

@antoinedang for some reason I can't request your review on this but if you could that would be great

@MrMondrian MrMondrian merged commit a135ec3 into noetic Sep 23, 2023
@MrMondrian MrMondrian deleted the new-state-estimation branch September 23, 2023 20:29
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.

3 participants