search

evcc-io / evcc

Sonne tanken ☀️🚘
https://evcc.io
MIT License
3.59k stars 661 forks source link

Missverständnis Phasenumschaltung vs. Konfiguration #12013

Closed andreashm closed 9 months ago

andreashm commented 9 months ago

Describe the bug

Bis Version 0.123.9 lief es korrekt: Bei Druck auf Schieberegler konnte man sehen, ob 11kW oder 3,7kW angeschlossen waren (sprich: Ob die Box 1p oder 3p mit Strom versorgt wurde). Das wurde auch für die PV-Steuerung entsprechend berücksichtigt.

Seit Version 0.124.0 gibt es hier einen Bug: Die Box wird nicht mehr nach den angeschlossenen Phasen gefragt, stattdessen muss man über Optionsschaltflächen auswählen, ob sie 1p oder 3p angeschlossen ist. Auswirkungen hat das aber keine, es wird das Fahrzeug 3p geladen, wenn die Wallbox 3p angeschlossen ist. Ob eine falsche Auswahl Auswirkungen auf die PV-Steuerung hat, kann ich wetterbedingt leider derzeit noch nicht ermitteln. Die Anzeige ist aber fehlerhaft bis irreführend.

Im Bild sieht man die falsche Anzeige bei 1p-Anschluss der Wallbox, weil es nicht mehr erkannt wird. Fehlerhafte-Anzeige-Wallbox

Steps to reproduce

Schließe eine Energie-Control von Heideberg an und nutze Version 0.124.0

Configuration details

-

Log details

-

What type of operating system are you running?

Linux

Version

0.124.0

andig commented 9 months ago

Logfile fehlt, ebenso Config.

andig commented 9 months ago

/cc @naltatis

naltatis commented 9 months ago

@andreashm Was hattest du denn vorher bei dem Ladepunkt als phases eingetragen? Mit der neuen Version ist es in der Tat so, dass wir eine feste Phaseneinstellung als Konfiguration am Ladepunkt erwarten.

VolkerK62 commented 9 months ago

Mit der neuen Version ist es in der Tat so, dass wir eine feste Phaseneinstellung als Konfiguration am Ladepunkt erwarten.

Das beißt sich doch aber dann mit der "automatischen" Erkennung über die Phasenspannungen.

naltatis commented 9 months ago

Das beißt sich doch aber dann mit der "automatischen" Erkennung über die Phasenspannungen.

@andig @premultiply seid ihr da im Thema? Diese automatische Erkennung hatte ich bislang nicht auf dem Zettel. Wie bilden wir die im Code ab?

andig commented 9 months ago

Das ist https://github.com/evcc-io/evcc/blob/4c84322c5cacbf4ad5bce2a6ff1c371763563d44/core/loadpoint.go#L1319

VolkerK62 commented 9 months ago

Ich hab im UI phases: 3 definiert. Wie früher in der yaml. Über die Spannungen wird 1p erkannt. In /api/state:

naltatis commented 9 months ago

Zur UI: Hier kommunizieren wir nicht klar genug, welche Auswirkungen diese Angabe hat. Bei 1p3p Geräten kontrolliert man damit direkt das Verhalten der Wallbox (auto, 3p, 1p). Bei "normalen" Chargern ist das lediglich die Information für evcc wie die Wallbox angeschlossen ist (3p o. 1p). Falschangabe führt zu keiner Verhaltensänderung, aber ggf. zu anderen internen Berechnungen. Das war in der yaml Einstellung (phases: 0/3/1) bislang genau so "überladen". Hier in der UI fällt es aber deutlicher auf.

Kurzfristige Maßnahme: Unterstützt die Box keine Phasenumschaltung zeigen wir einen Hinweistext, an der klarmacht, dass das hier lediglich eine Info für evcc ist.

Mittelfristige Maßnahme: Für Boxen ohne Phasenumschaltung wird diese Angabe ins Config UI (für Ersteinrichtung) verschoben. Voraussetzung ist, dass wir Ladepunkte/Charger dort anlegen können. Boxen mit Phasenumschaltung behalten die Phasenänderungsoptionen im Main UI (heutiger Ort).

VolkerK62 commented 9 months ago

ABER: Vor den Update wurde im UI die Ladeleistung bei mincurrent und maxcurrent richtig angezeigt, nämlich auf Basis der erkannten Phasen. Jetzt wird dafür wohl der eingestellte Phasenwert genommen.

naltatis commented 9 months ago

@VolkerK62 Ja, du hast recht: https://github.com/evcc-io/evcc/pull/11571/files#diff-3b9fd87655a9ea0010ac8a5b5128979d0fc96df5a48831983faddc6899c166bbL253

Ich schau mir das mal an.

andreashm commented 9 months ago

Hi, ich habe glaube keine phases in der yaml angegeben, weil ich im Schaltschrank manuell von 3p auf 1p umschalten kann. Ich kann gerade leider nicht nachschauen.

andig commented 9 months ago

Keine angegeben bedeutet 3p per default und dann läuft die Erkennung.

naltatis commented 9 months ago

Standardwert war bislang 3 (siehe Doku). Das hat sich mit der Umstellung ins UI auch nicht verändert. Daher sollte es "nur" das Darstellungsproblem sein was hier gefixt ist. 👆

Tombra1889 commented 9 months ago

Ich fände es in diesem Kontext optimal, wenn ich in dem Wertebereich nur die Werte auswählen könnte, die in der yaml hinterlegt sind:

image

Meine Wallbox unterstützt nur eine Leistung bis 16 A. Dieses habe ich auch so hinterlegt, auswählen könnte ich aber trotzdem bis 32 A.

VolkerK62 commented 9 months ago

Der Loadpoint-Eintrag in der yaml wird ignoriert. Die Einstellung findet jetzt im UI statt.

@andig @naltatis könnte/sollte man hier den Maxwert aus der Wallbox nehmen? Keine Ahnung, ob die von allen auslesbar sind.

naltatis commented 9 months ago

Ja, das kommt mit dem nächsten Schritt in der Config UI. Da würden dann harte min/max Werte definiert werden und hier (Settings am Loadpoint) nur noch innerhalb der Grenzen für den Alltag umstellbar.

Bis dahin ist dieser Dialog das globale Min/Max für den Ladepunkt (so wie voher die yaml war.)

naltatis commented 9 months ago

Ich hab die Doku entsprechend aktualisiert: https://github.com/evcc-io/docs/pull/522