Skip to content

Improve active bat control#3395

Open
ndrsnhs wants to merge 5 commits into
openWB:masterfrom
ndrsnhs:improve-active-bat-control
Open

Improve active bat control#3395
ndrsnhs wants to merge 5 commits into
openWB:masterfrom
ndrsnhs:improve-active-bat-control

Conversation

@ndrsnhs
Copy link
Copy Markdown
Contributor

@ndrsnhs ndrsnhs commented May 27, 2026

Die erlaubte Entladeleistung des Speichers (Speicherbeachtung) wird ignoriert sobald die aktive Speichersteuerung genutzt wird.
So wird verhindert, dass die Ladeleistung zu hoch eingestellt wird (Speicherentladung soll ja aktiv begrenzt werden)

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Diese PR soll verhindern, dass die konfigurierte “erlaubte Entladeleistung” (bat_power_discharge) bei aktivierter Speichersteuerung zusätzlich in die verfügbare Ladeleistung einfließt (und dadurch die resultierende Ladeleistung zu hoch ausfällt).

Changes:

  • Einführung einer Erkennung “aktive Speichersteuerung” in _get_charging_power_left(), um die Berücksichtigung von bat_power_discharge_active in diesem Fall zu unterdrücken.
  • Anpassung der Berechnung von charging_power_left in zwei SoC/Hysterese-Zweigen.

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

Comment thread packages/control/bat_all.py Outdated
Comment on lines +387 to +390
if self.data.set.power_limit is None:
if config.bat_power_discharge_active:
active_bat_control = (self.data.config.bat_control_permitted and
self.data.config.bat_control_activated)
if config.bat_power_discharge_active and not active_bat_control:
Comment thread packages/control/bat_all.py Outdated
Comment on lines +405 to +408
if self.data.set.power_limit is None:
if config.bat_power_discharge_active:
active_bat_control = (self.data.config.bat_control_permitted and
self.data.config.bat_control_activated)
if config.bat_power_discharge_active and not active_bat_control:
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread packages/control/bat_all.py Outdated
else:
if self.data.set.power_limit is None:
if config.bat_power_discharge_active:
if config.bat_power_discharge_active and not self.data.config.bat_control_activated:
@ndrsnhs ndrsnhs requested a review from LKuemmel May 27, 2026 09:53
@cr0i
Copy link
Copy Markdown
Contributor

cr0i commented May 28, 2026

Die erlaubte Entladeleistung des Speichers (Speicherbeachtung) wird ignoriert sobald die aktive Speichersteuerung genutzt wird. So wird verhindert, dass die Ladeleistung zu hoch eingestellt wird (Speicherentladung soll ja aktiv begrenzt werden)

Ich gehe davon aus, dass das dann genau entgegen dem Verhalten funktioniert, das man eigentlich haben möchte. Ich hatte das Problem hier gemeldet:
https://forum.openwb.de/viewtopic.php?p=142049#p142049
Es wird im Log zwar behauptet, dass die Entladeleistung nicht gesteuert wird, da mit vorgegebener Leistung entladen wird, es wird dann aber doch die Entladeleistung gesteuert.
Dadurch wird in diesem Fall statt mit der erlaubten Entladeleistung ungefähr mit dem Hausverbrauch entladen, was nicht Sinn der Übung sein sollte.

Es wäre Zeit für eine Grundsatzdiskussion hierzu, da das Problem schon von einigen bemerkt wurde.
Die aktive Speichersteuerung soll beim Laden mit Netzbezug den Speicher schonen. Es soll aber nicht jede Wolke beim Überschussladen zu Netzbezug führen, gerade wenn ich die passive Speichersteuerung nutze.
Ich gehe daher davon aus, dass die Funktion wie sie im Log gemeldet wird, die richtige wäre und nicht die erlaubte Entladung zu ignorieren.
In diesem Zusammenhang wurde noch eine obere Entladeschranke ins Spiel gebracht, die das Problem auch lösen könnte:
https://forum.openwb.de/viewtopic.php?p=142593#p142593

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