-
Notifications
You must be signed in to change notification settings - Fork 24
eskf with sensor concept, less states, and latest fixes from dev/auv-navigation-eskf #640
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
base: main
Are you sure you want to change the base?
Conversation
|
Your branch is behind the main branch, and that is causing merge conflicts. Update that and make sure all tests pass 😄 Also, you could still change the target branch to Talha's branch so the PR can be continued there, but it probably doesnt matter if this is up to date. |
…ses msg imu/data_raw and /orca/pose as the dvl info
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
…don't put the definition in the header
…(removed local utils, fixed naming)
684658e to
d87003b
Compare
for more information, see https://pre-commit.ci
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #640 +/- ##
==========================================
- Coverage 35.58% 33.08% -2.50%
==========================================
Files 38 42 +4
Lines 2254 2427 +173
Branches 686 689 +3
==========================================
+ Hits 802 803 +1
- Misses 1273 1445 +172
Partials 179 179
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
Hi, I did the following modifications:
btw I didn't branch from Talha's because I had already made lots of modifications, it was easier to implement the latest missing fixes from his branch in here, I hope this is okay:)
P.S: