kaidokert / rm32 / 25186753480

61%

main: 63%

Build Type

Pull #15

github

Committed by kaidokert

Commit Message

Extract shared MainState constructor and apply_motor_config

MainState was manually constructed with 30+ fields in both firmware
(main.rs) and harness (harness.rs), creating init divergence risk
flagged by reviewers on PRs 4, 5, 6, and 7.

Add MainState::new(MainStateParams) that takes the 7 board/chip
parameters that actually differ between contexts. Both firmware and
harness now call this single constructor.

Add MainState::apply_motor_config() to share the EEPROM-derived
PID/KV/voltage config application that was also duplicated.

Pull Request Pull Request #15: Extract shared MainState constructor and apply_motor_config

Coverage Stats

0 of 54 new or added lines in 2 files covered. (0.0%)

1 existing line in 1 file now uncovered.

3248 of 5314 relevant lines covered (61.12%)

592.96 hits per line