dangernoodle-io / TaipanMiner / 25297649943

90%

push

github

feat(mining): feed max(die, vr) into autofan PID (TA-141) (#320)

* feat(mining): feed max(die, vr) into autofan PID (TA-141)

Implement VR-aware thermal control: independently filter die and VR
temperatures via EMA (alpha=0.2), feed max(die_ema, vr_ema) to the PID
controller, and track which sensor is driving the fan. Addresses bitaxe-650
thermal gap where VR can throttle while die-only PID looks cool.

- Add s_die_ema, s_vr_ema, s_pid_input_c, s_pid_input_src statics in asic_task.c
- Apply independent EMA to each temp; skip VR update if sensor invalid (-1 sentinel)
- Compute pid_input = max(die_ema, vr_ema) with source tracking
- Expose asic_task_get_autofan_telemetry() accessor for /api/fan
- Add die_ema_c, vr_ema_c, pid_input_c, pid_input_src fields to fan_snapshot_t
- Update /api/fan response schema in routes.c
- Add 6 tests covering: die>vr, vr>die, vr invalid, independent EMA updates,
  source switching, initialization
- Single PID setpoint; no new config tunings

Bitaxe-650 context: EMC2101-R confirmed single-diode (TA-141 probe);
chip 1 unmonitored but VR is shared rail and now feeds fan control.
Related: TA-315 (PID implementation), TA-317 (calibration), TA-318 (OC watchdog).

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

* test(routes_json): cover sentinel-null branches in build_fan_json (TA-141)

Add test_fan_thermal_sentinels_null to verify die_ema_c, vr_ema_c, and pid_input_c
emit null JSON values when set to sentinel (-1.0f). Fixes coverage regression on PR #320.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>

713 of 858 branches covered (83.1%)

Branch coverage included in aggregate %.

10 of 10 new or added lines in 1 file covered. (100.0%)

1619 of 1666 relevant lines covered (97.18%)

247785.62 hits per line