Skip to content

chargepoint peaks: fix cp sum in case of error#3585

Merged
LKuemmel merged 1 commit into
openWB:masterfrom
LKuemmel:fixes
Jun 23, 2026
Merged

chargepoint peaks: fix cp sum in case of error#3585
LKuemmel merged 1 commit into
openWB:masterfrom
LKuemmel:fixes

Conversation

@LKuemmel

@LKuemmel LKuemmel commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Wenn Komponenten im Fehlerzustand sind, wird deren Zählerständ trotzdem in die Gesamtsumme eingerechnet, damit es keinen großen Sprung gibt und man zuverlässig eine Gesamtsumme hat. Wenn zB ein LP nicht funktioniert, fehlt dieser sonst in der Gesamtsumme und im nächsten Zyklus ist er wieder eingerechnet und es gibt einen Sprung.

Ticket #66005609

@LKuemmel LKuemmel added this to the 2.3.0 milestone Jun 23, 2026
@LKuemmel LKuemmel requested a review from Copilot June 23, 2026 08:52

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.

Pull request overview

Die Änderungen sorgen dafür, dass Gesamtwerte (v. a. Zählerstände) von PV-/Batterie-/Ladepunkt-Modulen auch dann weiter aufsummiert werden, wenn einzelne Komponenten im Fehlerzustand sind, um Sprünge/fehlende Gesamtsummen zu vermeiden (Ticket #66005609).

Changes:

  • Aggregationslogik für PV (pv_all.py), Batterie (bat_all.py) und Ladepunkte (chargepoint_all.py) so angepasst, dass Summen auch bei Fehlerzuständen der Einzelmodule weiter gebildet werden.
  • Fault-State-Ermittlung vereinheitlicht (per max(...)) und Fault-Strings entsprechend angepasst.
  • Neuer Test für AllChargepoints.get_cp_sum() ergänzt, der Summierung auch bei fehlerhaftem Ladepunkt abdeckt.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
packages/control/pv_all.py PV-Summenbildung angepasst (Power/Exported/Fault-State) und Publishes reorganisiert.
packages/control/chargepoint/chargepoint_all.py Ladepunkt-Summenbildung angepasst, sodass Zählerstände auch bei Fehlerzustand einfließen.
packages/control/chargepoint/chargepoint_all_test.py Neuer Pytest, der Summenbildung inkl. Fault-State/Fault-String prüft.
packages/control/bat_all.py Batterie-Summenbildung angepasst (Power/Import/Export/SoC/Fault-State).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/control/chargepoint/chargepoint_all.py
Comment thread packages/control/pv_all.py
Comment thread packages/control/bat_all.py
@LKuemmel LKuemmel merged commit 9364bfd into openWB:master Jun 23, 2026
1 check passed
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.

2 participants