PV-Laden: bei ausreichend Überschuss direkt mehrphasig statt einphasig starten - #3899
Open
seaspotter wants to merge 1 commit into
Open
PV-Laden: bei ausreichend Überschuss direkt mehrphasig statt einphasig starten#3899seaspotter wants to merge 1 commit into
seaspotter wants to merge 1 commit into
Conversation
Der Hilfetext zur Pufferzeit zwischen automatischen Phasenumschaltungen verspricht das schon, es gab dafür aber keine Implementierung beim Ladestart.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hintergrund
Der Hilfetext zur Einstellung "Pufferzeit zwischen automat. Phasenumschaltungen"
verspricht explizit:
Das hat aber tatsächlich nie funktioniert – es gab dafür an der Stelle,
an der beim Ladestart die Phasenzahl im Automatikmodus festgelegt wird, keine
Überschussprüfung. Es wurde unabhängig vom Überschuss immer mit 1 Phase
gestartet.
In der Praxis führte das sogar zu einem doppelten Nachteil, nicht nur zu einem
verzögerten Hochschalten: Stand der Ladepunkt hardwareseitig noch auf 3 Phasen
(z. B. vom vorherigen Ladevorgang), wurde beim Start unnötig auf 1 Phase
heruntergeschaltet. Diese Umschaltung setzt den Zeitstempel für die
Umschaltverzögerung zurück, wodurch anschließend – obwohl der Überschuss die
ganze Zeit ausgereicht hätte – erneut die volle Pufferzeit (Standard 7 Minuten)
abgewartet werden musste, bevor wieder auf 3 Phasen hochgeschaltet wurde.
Konkret beobachtet: Ladepunkt wurde gestartet, obwohl bereits ausreichend
PV-Überschuss vorhanden war. Es folgte dennoch eine unnötige Umschaltung
3→1 Phase und danach eine Statusmeldung "Umschaltung von 1 auf 3 Phasen in
7 Min. 41 Sek.", bevor tatsächlich mehrphasig geladen wurde.
Interessanterweise existiert die "direkt mehrphasig starten"-Logik an einer
anderen Stelle im Code bereits (
switch_on_timer_expiredincounter.py,erkennbar an der bisher toten Meldung
SWITCH_ON_MAX_PHASES) – sie greiftaber nur auf einem von mehreren Pfaden zum Ladestart, nicht dort, wo die
Phasenzahl im Automatikmodus grundsätzlich festgelegt wird.
Änderung
In
get_phases_by_selected_chargemode()wird beim Ladestart imAutomatikmodus (PV-Laden,
phases_to_use == 0) jetzt geprüft, ob dernutzbare Überschuss bereits für maximale Phasenzahl bei Mindeststromstärke
ausreicht. Ist das der Fall, wird direkt mit der maximalen Phasenzahl
begonnen, wie es der Hilfetext beschreibt. Andernfalls bleibt das bisherige
Verhalten (Start mit 1 Phase, Hochschalten über die reguläre
Umschaltverzögerung) unverändert.
Tests
get_phases_test.pyfür: ausreichend Überschuss (→ max.Phasen), nicht ausreichend Überschuss (→ 1 Phase), fest eingestellte
Phasenzahl (kein Überschusscheck), sowie Lademodus ungleich PV-Laden (kein
Überschusscheck).
packages/control/Testsuite (424 Tests) läuft weiterhin grün durch.flake8clean.