Skip to content

fix Counter: Fehlerfall erst nach 60s - #3951

Merged
LKuemmel merged 1 commit into
openWB:masterfrom
seaspotter:fix-counter-fehlerfall-timing
Sep 16, 2026
Merged

LKuemmel merged 1 commit into
openWB:masterfrom
seaspotter:fix-counter-fehlerfall-timing

Conversation

@seaspotter

@seaspotter seaspotter commented Sep 15, 2026 •

Copy link
Copy Markdown
Collaborator

Siehe: #3931 (comment)

  • Zähler-Fehlerfall (max. Leistung) griff sofort statt erst nach Ablauf der 60s-Wartezeit
  • RCT: connect_to_server() hat Verbindungsfehler verschluckt statt sie zu werfen, dadurch wurde fault_state fä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.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 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.

@LKuemmel

Copy link
Copy Markdown
Contributor

Lass uns vorher abstimmen, wie es werden soll, dann kannst Du das Verhalten gerne anpassen.
Soll es für PV und Speicher dann auch einen 60s Timer geben? bei den Ladepunkten gibt es sowas ja auch. Dann würde es für die Verbraucher auch noch Sinn machen.

@seaspotter

Copy link
Copy Markdown
Collaborator Author

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.

@LKuemmel

Copy link
Copy Markdown
Contributor

Sehr schön, dann merge ich diesen PR und für die anderen Komponenten können wir das dann in einem Issue besprechen.

@LKuemmel
LKuemmel merged commit 1591783 into openWB:master Sep 16, 2026
1 check passed
@seaspotter
seaspotter deleted the fix-counter-fehlerfall-timing branch September 19, 2026 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants