kaidokert / rm32 / 25191325120

62%

main: 63%

Build Type

Pull #18

github

Committed by kaidokert

Commit Message

Fix e_com_time: compute from ISR intervals, not empty main-loop array

The commutation_intervals array in TimingState was never populated by
the firmware ISR, only by the harness's manual copy. This meant
e_com_time, e_rpm, and average_interval were always zero in production.

Fix: move intervals array into Commutation struct (ISR-owned), add
record_interval() called at each step advance (matching C main.c:882),
and compute e_com_time in the ISR's sync section. Main loop now reads
e_com_time from SharedComm instead of computing it from a local array.

Remove the harness's manual intervals copy (no longer needed) and
the now-empty commutation_intervals field from TimingState.

Pull Request Pull Request #18: Fix e_com_time: compute from ISR intervals, not empty main-loop array

Coverage Stats

10 of 12 new or added lines in 3 files covered. (83.33%)

8 existing lines in 3 files now uncovered.

3271 of 5279 relevant lines covered (61.96%)

627.21 hits per line