Skip to content

Commit 23c44b5

Browse files
committed
Fix missed smooth_target_alpha in smoke test, update stale docstring
Remove smooth_target_alpha from CT002 constructor in smoke_efficiency_saturation.py (missed during refactor). Update test_balancer_probe_lockup.py docstring to remove reference to deleted test_smoother.py. https://claude.ai/code/session_01LrkYqzhz4EMqBAKG4YAdUR
1 parent 5d2b071 commit 23c44b5

2 files changed

Lines changed: 6 additions & 10 deletions

File tree

tests/smoke_efficiency_saturation.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -109,13 +109,13 @@ def __init__(
109109
ct_mac=ct_mac,
110110
active_control=True,
111111
fair_distribution=True,
112-
smooth_target_alpha=0.9,
113112
deadband=5,
114113
min_efficient_power=min_efficient_power,
115114
efficiency_rotation_interval=scaled_rotation,
116115
efficiency_saturation_threshold=efficiency_saturation_threshold,
117116
saturation_decay_factor=saturation_decay_factor,
118117
consumer_ttl=120 / time_scale,
118+
reset_fn=None,
119119
**ct_kwargs,
120120
)
121121

tests/test_balancer_probe_lockup.py

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -9,15 +9,11 @@
99
(visible in the log for ~1.5 hours until manual restart). The grid
1010
drifts ~97 W uncompensated for the entire window.
1111
12-
The root cause is exercised here at two levels:
13-
14-
1. A *unit-level* repro that drives :class:`LoadBalancer.compute_target`
15-
with the exact report sequence from the log (two phase-B consumers,
16-
scripted power outputs and meter readings) and asserts the active
17-
battery receives a reasonable target after the handoff.
18-
19-
2. A *smoother-level* repro in ``test_smoother.py`` covering the
20-
``sample_id`` dedup that fires even when ``raw_total`` has changed.
12+
The root cause is exercised here via a *unit-level* repro that drives
13+
:class:`LoadBalancer.compute_target` with the exact report sequence from
14+
the log (two phase-B consumers, scripted power outputs and meter
15+
readings) and asserts the active battery receives a reasonable target
16+
after the handoff.
2117
"""
2218

2319
from __future__ import annotations

0 commit comments

Comments
 (0)