Skip to content

fix: corrected continuous discrete update#60

Open
iandareid wants to merge 1 commit into
mainfrom
estimator-ekf-update
Open

fix: corrected continuous discrete update#60
iandareid wants to merge 1 commit into
mainfrom
estimator-ekf-update

Conversation

@iandareid
Copy link
Copy Markdown
Contributor

Fixed the continuous update according to @bsutherland333's analysis and retuned the estimator.

@iandareid iandareid self-assigned this May 18, 2026
@bsutherland333
Copy link
Copy Markdown
Member

That is technically correct, but your naming is a bit confusing. I personally would multiply the IMU process noise by the IMU timestep when you construct $Q_g$, meaning $Q_g = \sigma^2 T_s$, and then use $T_s (Q + G Q_g G^\top)$ for the propagation equation, where $T_s = T_{s, imu} / N$.

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