Skip to content

Fix iterated EKF measurement updates - #465

Open
Arthur031221 wants to merge 1 commit into
probml:mainfrom
Arthur031221:fix-iterated-update-double-count
Open

Arthur031221 wants to merge 1 commit into
probml:mainfrom
Arthur031221:fix-iterated-update-double-count

Conversation

@Arthur031221

@Arthur031221 Arthur031221 commented Oct 1, 2026 •

Copy link
Copy Markdown

The iterated EKF uses the previous posterior for each update, counting the same observation more than once. Increasing num_iter changes the result even for linear models. The fix keeps the original prior and corrects the linearization residual at each iteration.

Tested with the added linear and nonlinear regressions: .venv/bin/python -m pytest -q -k iterated dynamax/nonlinear_gaussian_ssm/inference_test.py.

Assisted by Claude/Codex.

Keep the original prior during relinearization so each measurement is
counted once. Test linear models and a nonlinear posterior mode.
@Arthur031221
Arthur031221 force-pushed the fix-iterated-update-double-count branch from 3ef4300 to 87d1a4d Compare October 12, 2026 00:05
@Arthur031221 Arthur031221 changed the title Fix iterated EKF and CMGF update counting the observation num_iter times Fix iterated EKF measurement updates Oct 12, 2026
@Arthur031221

Copy link
Copy Markdown
Author

Rebased onto main and moved the EKF regressions into the current test module. This PR now covers only the iterated EKF update because the CMGF covariance solve needs a separate numerical design.

The linear and nonlinear EKF regressions pass.

This branch has not been deployed

No deployments
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.

1 participant