Skip to content

feat(arm): integrate KalmanTracker into main control loop#1

Merged
Aiden Young (aiyds) merged 2 commits into
mainfrom
feat/kalman-tracker-integration
Jun 11, 2026
Merged

feat(arm): integrate KalmanTracker into main control loop#1
Aiden Young (aiyds) merged 2 commits into
mainfrom
feat/kalman-tracker-integration

Conversation

@ehuang2021

Copy link
Copy Markdown
Member

Wire kalman_tracker.h into the 100Hz control loop (Step 2b).
Confirmed tracks (age >= 3 frames) replace raw CollisionPredictor
output before EvasionController sees it, suppressing single-frame
noise from triggering evasive maneuvers.

  • Add tracker.update() + get_confirmed_tracks() after predictor.assess()
  • Rebuild assessment.objects with KF-filtered positions
  • Add trk=/conf= columns to log_telemetry for flight debugging
  • Add make test target + test_arm_cp.cpp (6 unit tests, all passing)
  • Fix missing kalman_tracker.h dependency in Makefile %.o rule

Fixed a sim segfault.

Subtract FPGA_BASE_ADDR before indexing into the registers array.
offset/4 was computing index ~285M into a 128-element array, causing
an immediate segfault in drone_control_sim. Also correct for real
hardware mmap path where registers_ points to the base of the mapping.
Wire kalman_tracker.h into the 100Hz control loop (Step 2b).
Confirmed tracks (age >= 3 frames) replace raw CollisionPredictor
output before EvasionController sees it, suppressing single-frame
noise from triggering evasive maneuvers.

- Add tracker.update() + get_confirmed_tracks() after predictor.assess()
- Rebuild assessment.objects with KF-filtered positions
- Add trk=/conf= columns to log_telemetry for flight debugging
- Add make test target + test_arm_cp.cpp (6 unit tests, all passing)
- Fix missing kalman_tracker.h dependency in Makefile %.o rule
@aiyds Aiden Young (aiyds) merged commit ede189b into main Jun 11, 2026
5 checks passed
@aiyds Aiden Young (aiyds) deleted the feat/kalman-tracker-integration branch June 11, 2026 18:56
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