Battery control: hold battery while min soc not reached

[rivengh]

This PR extents the battery control mechanism to hold the battery also while minSoc is not reached, corresponding to #11511.

From my point of view this case should be treated in the same way as when smartCostActive or api.ModeNow is true.

[premultiply]

Hatten wir schon mal diskutiert.
Fazit war: Nicht sinnvoll da unnötig teurer Netzbezug generiert wird.

[andig]

Seems you could do the same by physically configuring minsoc on your battery. Hence nothing we should do in software.

[rivengh]

Seems you could do the same by physically configuring minsoc on your battery. Hence nothing we should do in software.

No, this is a different story. I am referring to the vehicles minSoc not the home battery soc.
But anyway, it seems that you already discussed this topic and decided to don't include this case.

[andig]

I don't think this was discussed before, more that me (and probably @premultiply) misunderstood the intention of this PR.

@premultiply es geht um Sofortladen bis minSoc- den Aspekt hatten mir m.E. noch nicht diskutiert?

/cc @naltatis

[naltatis]

Mir ist das in der eigenen Nutzung auch schon aufgefallen. Glaube auch, dass wir? (aber auf jeden Fall ich) über Min-Soc einfach nicht gesprochen/nachgedacht haben. Eigentlich finde ich die Änderung hier konsistent mit unserer anderen "Schnell-Laden im PV-Modus" Funktionen. Also "weil Preis günstig" oder "weil Abfahrtszeit gesetzt". Da setzen wir ja auch Hold. Der Fall Schnelladen "weil Fahrzeug Akkustand niedrig" passt für mich genau in diese Reihe und wäre erwartungskonformes Verhalten.

@premultiply Aussage, dass das ggf. teurer ist, mag je nach Umständen stimmen. Aber daher machen wir "Batterycontrol" ja zu nem Opt-In Feature, für, dass sich der Nutzer aktiv entscheiden muss.

Kurz: ich wäre dafür, das wieder aufzumachen.

[andig]

LGTM. @premultiply? Hier geht es um Schnelladen während das Fahrzeug seinen minSoc nicht erreicht hat- also analog Sofortladen. Ok?

[premultiply]

Ich enthalte mich dazu. 🤷🏻‍♂️
Macht wenn ihr denkt dass das besser ist.

[andig]

Also doch kein Mißverständnis?

Fazit war: Nicht sinnvoll da unnötig teurer Netzbezug generiert wird.

Könntest Du nochmal sagen wo der Unterschied zu "Schnelladen" ist?

[premultiply]

Schnellladen ist etwas was man ggf. aktiv auswählen muss bzw. kann.
Das minSoc-Laden wird hingegen nachdrücklich automatisch zu jedem Zeitpunkt gestartet wenn das Fahrzeug nur angeschlossen wird und sonst im PV-Modus nicht geladen würde. Das kann auch bei sehr, sehr teurem Netzstrom sein.

Letztendlich ist es aber wahrscheinlich reine Geschmackssache.

[rivengh]

Danke, dass ihr den PR doch nochmal in Betracht zieht.

Ich sehe es genau so wie @naltatis geschrieben hat, dass es einfach konsistent zu den anderen Funktionen ist:

Eigentlich finde ich die Änderung hier konsistent mit unserer anderen "Schnell-Laden im PV-Modus" Funktionen. Also "weil Preis günstig" oder "weil Abfahrtszeit gesetzt". Da setzen wir ja auch Hold. Der Fall Schnelladen "weil Fahrzeug Akkustand niedrig" passt für mich genau in diese Reihe und wäre erwartungskonformes Verhalten.

Ich kann den Einwand von @premultiply natürlich auch verstehen, jedoch trifft, dass imho auf alles Schnelllade-Modi zu. Außerdem kann ja auch das minSoc-Laden vom Nutzer explizit aktiviert oder deaktiviert werden. Ist es aktiviert, lebe ich immer mit dem Risiko, dass nach dem Anstecken aus dem Netz geladen wird.

[GrimmiMeloni]

Sieht für mich auch schlüssig aus.
Ich nutze das Vehicle min-Soc Feature bisher nicht, daher hatte ich das nicht auf dem Radar. Für mich ist das aber genau das gleiche wie Modus "schnell". Sollten wir wie gezeigt mit reinnehmen.