Avoid offline vehicles charging if minSoc is set

[MarkusGH]

If minSoc is set for a offline vehicle, the loadpoint will start to charge as soon as the vehicle is selected in the UI.
Unfortunately minSoc cannot be changed in the UI for offline vehicles, so the user can't fix that by just setting minSoc to 0.

This PR fixes that by changing minSocNotReached, so it returns false if the vehicle does not provide SoC.

[andig]

Good catch. Was tut der zweite Teil der Änderungen?

[MarkusGH]

Da kam man nur an wenn das EV SoC == 0 meldet und minSoc > 0 ist.
Jetzt wird aber vorher schon geprüft, ob SoC ein valider Wert ist.
Deswegen kann man SoC == 0 genauso behandeln wie alle anderen Werte.
Habe es nochmals vereinfacht - die Prüfung auf minSoc == 0 ist auch obsolet.

[andig]

Hab ich noch nicht verstanden, braucht vielleicht mehr Ruhe dafür.

Ich habe aber noch ein Grundsatzproblem: was machen wir mit "normalen" Fahrzeugen die beim Neustart aus diversen Problemen plötzlich offline sind? Diese würden mit der Änderung nicht mehr geladen!

Im Zweifel votieren wir eigentlich immer für Energie im Fahrzeug... das spräche gegen diesen PR. Damit verbunden die Frage:

If minSoc is set for a offline vehicle, the loadpoint will start to charge as soon as the vehicle is selected in the UI.
Unfortunately minSoc cannot be changed in the UI for offline vehicles

Wie kann das denn dann passieren?

[MarkusGH]

Was bei mir passiert ist:
Ich habe ein "normales" EV, dafür wurde ein minSoc von 50% gesetzt.
Weil das Porsche API scheinbar mal wieder kaputt ist, ist das EV nach Neustart von evcc offline.
Leider ist der minSoc weiterhin gesetzt und wird im UI angezeigt.
Deswegen wird das Fahrzeug jetzt geladen bis es voll ist und nichts mehr annimmt.
Im UI kann ich minSoc nicht ändern, weil offline.
Und so wie ich das UI verstehe, ist minSoc für offline EVs nicht vorgesehen: "Hinweis: Diese Optionen sind nur für Fahrzeuge mit bekanntem Ladestand verfügbar."
Fazit: Wenn der SoC nicht bekannt ist macht das Anwenden von minSoc keinen Sinn. Das ist auch die Prämisse im UI.

[andig]

Fazit: Wenn der SoC nicht bekannt ist macht das Anwenden von minSoc keinen Sinn. Das ist auch die Prämisse im UI.

Es sei denn Dein Fahrzeug ist wg. API Problemen offline und Du willst morgen los fahren. Catch 22.

[andig]

Wäre es in dem Fall nicht sinnvoller, auf Gastfahrzeug umzuschalten und fertig? Nicht schön, aber dann dann wird im Fehlerfall wenigstens noch geladen...

[MarkusGH]

Wäre es in dem Fall nicht sinnvoller, auf Gastfahrzeug umzuschalten und fertig? Nicht schön, aber dann dann wird im Fehlerfall wenigstens noch geladen...

Wir diskutieren hier das Verhalten bezüglich minSoc.
Das Gastfahrzeug hat immer minSoc = 0, da wird genausowenig geladen weil ein minSoc gesetzt wäre.

Und wenn das Offline Fahrzeug angesteckt wird oder evcc neu startet landet man sowieso im Gastfahrzeug.

Ein Automatismus bezüglich minSoc kann nur funktionieren, wenn Fahrzeugerkennung an ist und der SoC des Fahrzeugs bekannt ist.

[andig]

Nochmal. Wenn ein "normales" Fahrzeug einen minsoc gesetzt hat dann sollte es auch geladen werden wenn das API ausfällt. Das ist das aktuelle Verhalten.

Das Gastfahrzeug hat immer minSoc = 0, da wird genausowenig geladen weil ein minSoc gesetzt wäre.

Ich dachte das ist genau das was Dich stört- dass Dein Auto plötzlich lädt. Daher Vorschlag: auf Gast umstellen.

Und wenn das Offline Fahrzeug angesteckt wird oder evcc neu startet landet man sowieso im Gastfahrzeug.

...es sei denn es wäre als Default konfiguriert.

[MarkusGH]

Nochmal. Wenn ein "normales" Fahrzeug einen minsoc gesetzt hat dann sollte es auch geladen werden wenn das API ausfällt. Das ist das aktuelle Verhalten.

Es wird dann ggf. auf 100% geladen - ist das tatsächlich gewünscht?

Das Gastfahrzeug hat immer minSoc = 0, da wird genausowenig geladen weil ein minSoc gesetzt wäre.

Ich dachte das ist genau das was Dich stört- dass Dein Auto plötzlich lädt. Daher Vorschlag: auf Gast umstellen.

Ich halte es für problematisch, dass ich in dem Fall eine nicht änderbare minSoc Einstellung habe, die das Verhalten steuert.
Für mich kein Problem, ich kann einen SQL Prompt bedienen.
Für den normalen User wird es in dem Fall schwer noch was zu ändern.

[andig]

Es wird dann ggf. auf 100% geladen - ist das tatsächlich gewünscht?

Wenn die Alternative ein leeres Fahrzeug ist- ja!

[MarkusGH]

Es wird dann ggf. auf 100% geladen - ist das tatsächlich gewünscht?

Wenn die Alternative ein leeres Fahrzeug ist- ja!

Schön - dann ist das aktuell zwar befremdlich aber das gewünschte Verhalten.

Dann sollte man aber die Darstellung im UI ändern ("Hinweis: Diese Optionen sind nur für Fahrzeuge mit bekanntem Ladestand verfügbar.") und das Ändern von minSoc auch für offline Fahrzeuge erlauben.

[andig]

Schön - dann ist das aktuell zwar befremdlich aber das gewünschte Verhalten.

Isso- aber noch blöder ist ein leeres Auto.

Dann sollte man aber die Darstellung im UI ändern ("Hinweis: Diese Optionen sind nur für Fahrzeuge mit bekanntem Ladestand verfügbar.") und das Ändern von minSoc auch für offline Fahrzeuge erlauben.

/cc @naltatis

[andig]

Closing since #12460 now is a tracking issue.