SMA-10: two idle states, hold 0 W or hand the inverter back #239

Merged

admin merged 4 commits from SMA-10 into release/1.0 2026-11-07 23:43:15 +01:00

Conversation 3 Commits 4 Files Changed 12

+1972 -21

4 Commits

Author	SHA1	Message	Date
glenn schrooyenandClaude	6cae5a2f8f	SMA-10 round 4: pin the Dutch default "de eigen sturing" ... The NL default for a hand-back driver without words (handback._BACK_NL) was changed to "de eigen sturing" in round 2 at the PR #239 review's request (comment 2495: the article as in the generic clause). The brief does not name it, so DECISIONS now says where it came from and a test checks the exact Dutch sentence. No app code change. Co-Authored-By: Claude <noreply@anthropic.com>	2026-11-07 22:45:08 +01:00
glenn schrooyenandClaude	313503724d	SMA-10 round 3: Start clears a failed hand-back streak; the hook is tested ... When control takes the unit again, release_fails, release_retry_at and release_zero_sent are reset, so the next Stop is a new streak with its own one 0 W, error line and event. Tests: a streak that Start interrupts without a recovery; a driver that overrides release_fallback(). _BACK_NL "de eigen sturing" (round 2) is kept. Co-Authored-By: Claude <noreply@anthropic.com>	2026-11-07 21:31:28 +01:00
glenn schrooyenandClaude	d26cc573fe	SMA-10 round 2: a failed release sends 0 W once, retries at heartbeat pace ... When release() fails (False, raises or hangs past RELEASE_BUDGET_S) the unit gets one bounded write(0) through the new Driver.release_fallback() hook (default write(0)), then nothing more, so the inverter's own fallback timer is not reset. The release is retried once per heartbeat, not on every idle write. One error line and one action-log event per failure streak, one line on recovery. Dutch default "terug naar de eigen sturing". DECISIONS says what a unit is left at. test_messages.py: the RELEASE_BUDGET_S data entry is gone again (the line that needed it is gone). Co-Authored-By: Claude <noreply@anthropic.com>	2026-11-07 20:17:01 +01:00
glenn schrooyenandClaude	a1be9c48b3	SMA-10: two idle states, hold 0 W or hand the inverter back ... A driver now says whether its release() hands the inverter back (Driver.hands_back; GoodWe is False, so release() stays write(0) and the Controller never calls it). command_site_release() sits next to command_site_zero(): the Stop button and shutdown() (so SIGTERM, restart, control task death and a late config refusal) hand a hand-back unit back, once, and zero every other unit as before. Safety stops and refusals still hold 0 W for every unit. Shadow mode releases a hand-back unit. After a release nothing is written to the unit (heartbeat and cycle included) until control starts again. release() is bounded by RELEASE_BUDGET_S (= the write budget), runs concurrently per unit, and a raising or hanging release on one unit does not stop the others. The "set to 0 W" sentences for Stop, restart, shadow mode and the stopped banner are built from the roster (app/handback.py, EN and NL, words from the driver); a GoodWe-only roster keeps every old literal. A handed-back unit shows "Back under <its control>" on Home. Existing test edited: test_messages.py (two classification entries for the CRITICAL line that moved into a roster builder, and the new release bound). Co-Authored-By: Claude <noreply@anthropic.com>	2026-11-07 18:21:28 +01:00