Skip to content

Fix deprecated timestamp indexing in EKF - #577

Merged
xoolive merged 1 commit into
xoolive:masterfrom
Ashwini8686:fix-ekf-timestamp-indexing
Oct 14, 2026
Merged

xoolive merged 1 commit into
xoolive:masterfrom
Ashwini8686:fix-ekf-timestamp-indexing

Conversation

@Ashwini8686

Copy link
Copy Markdown
Contributor

Summary

  • Replace deprecated integer-key indexing with ".iloc" for positional access to timestamps in "extended_kalman_filter".
  • Preserve the existing time-difference calculation.

Testing

  • "uv run pytest tests/test_filter.py -q — 2 passed", 1 skipped.
  • "uv run ruff check src/traffic/algorithms/filters/ekf.py "— passed.

@Ashwini8686

Copy link
Copy Markdown
Contributor Author

Hi maintainers! Thank you for taking the time to look at my contribution. I've submitted this fix and verified it with the relevant tests and Ruff. When you get a chance, could you please review my PR and let me know if there are any suggestions or changes needed? I'd be happy to make improvements. Thank you!

@xoolive
xoolive merged commit 1d34bad into xoolive:master Oct 14, 2026
4 of 7 checks passed
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