Skip to content

fix: min_soc_bat_mode - #4116

Merged
LKuemmel merged 1 commit into
openWB:masterfrom
seaspotter:fix-bat-min-soc-discharge-not-allowed
Oct 16, 2026
Merged

LKuemmel merged 1 commit into
openWB:masterfrom
seaspotter:fix-bat-min-soc-discharge-not-allowed

Conversation

@seaspotter

Copy link
Copy Markdown
Collaborator

Speicher-Leistung bei Entladung-nicht-erlaubt nur bei echter Entladung kappen

#3908 (comment)

# Fahrzeug weitergereicht werden, sonst wird der Speicher entladen, obwohl der
# Nutzer das nicht erlaubt hat. Lädt der Speicher dagegen, bleibt diese Leistung
# weiterhin für ihn reserviert.
charging_power_left = max(base_power, 0)

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.

Das heißt, oberhalb des Max-SoCs gibt es kein eindeutigen Fahrzeug oder Speicher-Vorrang mehr, sondern first come first served?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Ja, richtig beobachtet - aber keine neue Eigenschaft von diesem PR. Im Band (hysteresis_discharge=True) gab es noch nie eine feste Priorität zwischen Fahrzeug und Speicher; charging_power_left spiegelt einfach, was der Speicher gerade tut. Vor #3986 war das ja schon = base_power und mit #3986 hatte ich das in diesem Zweig versehentlich auf 0 gesetzt, wodurch auch die berechtigte Ladereservierung für den Speicher verloren ging. Dieser PR stellt nur das Vorzeichen-Verhalten wieder her, ändert aber nichts an der fehlenden festen Priorität selbst.

Ob der Speicher im Band einen festen Vorrang bekommen soll, symmetrisch zu #3986 beim Aufstieg, ist strukturell identisch mit #3985 (der ist ja noch offen) und dort schon mit @ndrsnhs offen diskutiert, ohne Entscheidung. Ich würde #4116 unabhängig davon als Bugfix mergen - die Vorrang-Frage weiter auf #3985/#3980 klären.

@LKuemmel
LKuemmel merged commit a8bc5c3 into openWB:master Oct 16, 2026
1 check passed
@seaspotter
seaspotter deleted the fix-bat-min-soc-discharge-not-allowed branch October 20, 2026 18:57
LKuemmel added a commit to LKuemmel/core that referenced this pull request Oct 21, 2026
…bt nur bei echter Entladung kappen (openWB#4116)"

This reverts commit a8bc5c3.
LKuemmel added a commit that referenced this pull request Oct 21, 2026
* Revert "min_soc_bat_mode: Speicher-Leistung bei Entladung-nicht-erlaubt nur bei echter Entladung kappen (#4116)"

This reverts commit a8bc5c3.

* Revert "Speicher-Vorrang bei Erholung von Mindest- bis Maximal-SoC (#3986)"

This reverts commit 606cf2b.
LKuemmel added a commit that referenced this pull request Oct 22, 2026
* Revert "min_soc_bat_mode: Speicher-Leistung bei Entladung-nicht-erlaubt nur bei echter Entladung kappen (#4116)"

This reverts commit a8bc5c3.

* Revert "Speicher-Vorrang bei Erholung von Mindest- bis Maximal-SoC (#3986)"

This reverts commit 606cf2b.
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