Repository navigation
fix Counter: Fehlerfall erst nach 60s - #3951
Conversation
…hler nicht mehr verschluckt
There was a problem hiding this comment.
🟢 Approval recommended
No unresolved blocking issues were identified.
Pull request overview
Fixes RCT connection fault propagation and ensures the counter’s 60-second error grace period is respected.
Changes:
- Propagates RCT connection failures.
- Corrects counter grace-period handling.
- Adds regression coverage.
File summaries
| File | Description |
|---|---|
packages/modules/devices/rct/rct/rct_lib.py |
Propagates connection failures. |
packages/control/counter.py |
Corrects grace-period state handling. |
packages/control/counter_test.py |
Tests grace-period availability. |
Review details
- Files reviewed: 3/3 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
Lass uns vorher abstimmen, wie es werden soll, dann kannst Du das Verhalten gerne anpassen. |
|
Ich denke es macht Sinn den gleichen 60sec Timer zu nehmen wie für den Counter. Wobei es wird ja soweit ich das verfolgt habe auch gar nicht mehr weiter mit den Werten gerechnet, sie frieren dann nur ein. Ich schau mir das die Tage nochmal an, ist ja doch etwas komplexer und mach mal einen Vorschlag. |
|
Sehr schön, dann merge ich diesen PR und für die anderen Komponenten können wir das dann in einem Issue besprechen. |
Siehe: #3931 (comment)
connect_to_server()hat Verbindungsfehler verschluckt statt sie zu werfen, dadurch wurdefault_statefälschlich zwischenzeitlich wieder auf "kein Fehler" gesetzt, obwohl die Verbindung durchgehend down war@LKuemmel Das Verhalten das im Fall eines Verbindungsfehlers die Werte von bat und inverter "einfrieren" ist das n Thema was irgendwann nochmal angegangen werden soll oder soll das so bleiben? Also power auf 0 setzen z.B. dann in so einem Fall. Da war ich mir unsicher.