kaidokert / rm32

62%

main: 63%

LAST BUILD ON BRANCH pr12_1

branch: pr12_1

CHANGE BRANCH

x

Reset

Sync Branches

• pr12_1
• main
• pr1
• pr10_1
• pr11_1
• pr13_1
• pr14_1
• pr15_1
• pr16_1
• pr17_1
• pr18_1
• pr19_1
• pr20_1
• pr21_1
• pr22_1
• pr23_1
• pr24_1
• pr25_1
• pr26_1
• pr27_1
• pr28_1
• pr29_1
• pr2_1
• pr2_2
• pr2_3
• pr2_4
• pr2_5
• pr2_6
• pr2_7
• pr2_ref
• pr30_1
• pr31_1
• pr32_1
• pr33_1
• pr34_1
• pr3_1
• pr5_1
• pr7_1
• pr8_1
• pr_codecov1
• pr_codecov2
• system-tick
• tmps

Build # 25191753470

Build Type

Pull #18

github

Committed by kaidokert

Commit Message

Move e_com_time computation from per-tick to per-step

Computing the 6-interval sum on every 20kHz ISR tick was wasteful —
it only changes when a commutation step advances. Move the computation
into record_interval() which returns the updated e_com_time, and
publish to SharedComm at each of the 3 advance() call sites.

record_interval after advance matches C ordering where step is
incremented before commutation_intervals[step-1] is written.

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

Coverage Stats

13 of 17 new or added lines in 3 files covered. (76.47%)

8 existing lines in 3 files now uncovered.

3274 of 5284 relevant lines covered (61.96%)

596.34 hits per line