Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Min.Ladung% lässt sich nicht ändern wenn Fahrzeug Offline #12460

Closed
gennergy opened this issue Feb 25, 2024 · 16 comments · Fixed by #12841
Closed

Min.Ladung% lässt sich nicht ändern wenn Fahrzeug Offline #12460

gennergy opened this issue Feb 25, 2024 · 16 comments · Fixed by #12841
Assignees
Labels
enhancement New feature or request

Comments

@gennergy
Copy link

gennergy commented Feb 25, 2024

Describe the bug

Der Porsche Server schmeißt aktuell wieder Internal Server Error:
[main ] ERROR 2024/02/25 11:52:33 creating vehicle taycan failed: cannot create vehicle type 'template': cannot create vehicle
type 'porsche': unexpected status: 500 (Internal Server Error)
.Soc: vehicle not available: cannot create vehicle type 'template': cannot create vehicle type 'porsche': unexpected status:
500 (Internal Server Error)
Capacity: 84.0kWh
Features: [Offline]

Das Problem ist, dass nun das Fahrzeug sofort beim einstecken geladen wird, weil ein Wert für die mindest-Ladung eingestellt ist. Stand der Wert vor dem Offline Modus des Fahrzeugs auf --, könnte nun PV Regeln geladen werden. Leider kann der "Min. Ladung %" Wert nicht verändert werden, wenn das Fahrzeug Offline ist.

Steps to reproduce

  1. Fahrzeug online
  2. Min. Ladung % auf Wert >0% setzen
  3. Fahrzeug offline
  4. Min. Ladung % kann nicht mehr geändert werden.

Configuration details

nebensächlich

Log details

kein Log, Gui Problem

What type of operating system are you running?

Docker container

Version

evcc version 0.124.4

@andig
Copy link
Member

andig commented Feb 25, 2024

Das Problem ist, dass nun das Fahrzeug sofort beim einstecken geladen wird, weil ein Wert für die mindest-Ladung eingestellt ist.

Das ist Absicht. Die Alternative wäre nicht zu laden- damit stehst Du dann schlimmstenfalls im leerem Auto da.

Das Problem ist, dass nun das Fahrzeug sofort beim einstecken geladen wird

Das kann nur passieren, wenn Du ein Defaultfahrzeug konfiguriert hast oder dieses per RFID (oder ISO) einloggst- anderenfalls würde es beim Anstecken mangels API gar nicht erkannt.

Workaround: Defaultfahrzeug aus der Config nehmen oder Fahrzeug auf Gast stellen.

/cc @naltatis @MarkusGH

@andig andig added the wontfix This will not be worked on label Feb 25, 2024
@MarkusGH
Copy link
Contributor

@andig: Du hast das "wontfix" Label hinzugefügt. Wollt Ihr das UI Problem nicht korrigieren?

@StefanSchoof
Copy link
Contributor

Ich gebe andig Recht, das im Fehlerfall laden besser ist. Aber ich verstehe nicht warum man nicht in der Lage sein sollte den minSoc wieder per GUI raus zu nehmen. Wenn man auf Gastfahrzeug wechselt, wird die Lademenge in der Zwischenzeit nicht auf das richtige Fahrzeug gespeichert.

@gennergy
Copy link
Author

gennergy commented Feb 25, 2024

Ich nutze die mindest Ladung nur um ein tiefentladenen Akku wieder auf auf eine wie gesagt Mindest Ladung zu bringen, damit er nicht solange entladen rumsteht bis die Energie günstig ist. Den Ladestand habe ich immer soweit im Auge, dass ich weiß, wenn ich "Schnell" laden muss.
Das Fahrzeug ist im Monent 60% geladen. Damit komme ich in der Regel 2-3 Tage hin. Am Wochenende ist Tibber günstig und ich lade meist für die ganze Woche. Ich sehe jetzt keine Chance, automatisiert den günstigen Preis abzugreifen. Es sei denn, ich könnte jetzt bei dem Offline Fahrzeug die mindest Ladung wegdrehen. Ich würde eher auf die Mindest Ladung verzichten und es vorsorglich abstellen.
Denn gerade bei dem Porsche ist doch dauernd ein Problem mit dem Server oder dem Login. (Deswegen ist wahscheinlich das Tibber Porsche "Power Up" seit über einem Jahr nicht gefixt)

@andig
Copy link
Member

andig commented Feb 25, 2024

Das mag alles sein, ist aber Dein UC. Wir handeln grundsätzlich so, dass laden Vorrang vor minimalem Energieverbrauch hat.

@MarkusGH
Copy link
Contributor

Das mag alles sein, ist aber Dein UC. Wir handeln grundsätzlich so, dass laden Vorrang vor minimalem Energieverbrauch hat.

Das ist doch unbestritten - aber in dem Issue hier geht es doch nur um das UI, das einen eine einmal eingestellte und persistierte Mindestladung nicht mehr ändern lässt wenn ein Fahrzeug "offline" geht.

@andig
Copy link
Member

andig commented Feb 25, 2024

Da sind wir uns seit ungefähr 10 Kommentaren einig...

Es sei denn, ich könnte jetzt bei dem Offline Fahrzeug die mindest Ladung wegdrehen. Ich würde eher auf die Mindest Ladung verzichten und es vorsorglich abstellen.

Wie das geht hab ich ja oben beschrieben. Keine Ahnung, warum wir uns da im Kreis drehen?

@MarkusGH
Copy link
Contributor

MarkusGH commented Feb 25, 2024

Es sei denn, ich könnte jetzt bei dem Offline Fahrzeug die mindest Ladung wegdrehen. Ich würde eher auf die Mindest Ladung verzichten und es vorsorglich abstellen.

Wie das geht hab ich ja oben beschrieben. Keine Ahnung, warum wir uns da im Kreis drehen?

Kann sein ich bin begriffsstutzig, aber wie kann ich bei einem "offline" Fahrzeug einen zuvor gesetzten und damit in der Konfiguration gespeicherten Wert für die Mindestladung wieder löschen?

@MarkusGH
Copy link
Contributor

MarkusGH commented Feb 25, 2024

@gennergy: Korrigiere bitte mal Deine "Steps to reproduce:"

  1. Fahrzeug online
  2. Min. Ladung % auf Wert >0% setzen
  3. Fahrzeug offline
  4. Min. Ladung % kann nicht mehr geändert werden.

Vielleicht hilft das beim Verständnis dessen um was es hier geht.

@naltatis
Copy link
Member

@andig Wir müssen hier nochmal überdenken, wie wir mit Online-Fahrzeugen umgehen, die aus irgendeinem Grund nicht mehr erreicht werden können. Passiert das während der Laufzeit von evcc (Hersteller-Server nicht erreichbar) haben wir aktuell ein anderes Verhalten, als wenn das bei einem Neustart auftritt. Diese Unterschiede sind ja auch der Grund für den hier beschriebenen komischen Zustand.

Der Fahrzeug-bei-Neustart-Offline Fall führt dazu, dass der Nutzer ein Fahrzeug sieht, was zu meinen einen anderen Titel hat (+ Offline), das macht auch Quatscheinträge im Ladelog. Das größere Problem ist aber, dass wir in diesem Zustand einen Daten-Zwitter aus Offline-/Online-Fahrzeug in der UI und Steuerung bekommen. Offlinefahrzeuge haben weder MinSoc noch Plan-Soc. Ein Onlinefahrzeug das durch Neustart offline wird, kann diese beiden Werte aktuell aber haben. Hier müssen wir entweder diese Felder bei der Initialisierung überspringen oder (wäre mein Favorit) einen separaten "aktuell nicht erreichbar" Status am Fahrzeug setzen. Auf der Basis könnte die UI dann ne Markierung (bspw. Offline Label) hinzufügen. Die Funktionen in der UI würde aber weiterhin die eines Online-Fahrzeuges (evtl. mit Warnhinweis versehen) bleiben. Das Ladeverhalten (im zweifelsfall bei minsoc laden) würde dann genau so bleiben wie jetzt, nur dass man es weiterhin editieren kann.

@andig
Copy link
Member

andig commented Mar 9, 2024

Der Fahrzeug-bei-Neustart-Offline Fall führt dazu, dass der Nutzer ein Fahrzeug sieht, was zu meinen einen anderen Titel hat (+ Offline), das macht auch Quatscheinträge im Ladelog.

Das ist blöd, lässt sich aber anschließend korrigieren.

Das größere Problem ist aber, dass wir in diesem Zustand einen Daten-Zwitter aus Offline-/Online-Fahrzeug in der UI und Steuerung bekommen. Offlinefahrzeuge haben weder MinSoc noch Plan-Soc. Ein Onlinefahrzeug das durch Neustart offline wird, kann diese beiden Werte aktuell aber haben.

Das stimmt. Dieses Fahrzeug kann aber per Erkennung nie gefunden werden- denn es ist ja offline und hat keinen Status. In den Grenzfall kann man daher nur kommen wenns ein Defaultfahrzeug ist. Und dann halte ich es erstmal für korrekt- mangels besseren Wissens- das Fahrzeug zu laden.

Hier müssen wir entweder diese Felder bei der Initialisierung überspringen oder (wäre mein Favorit) einen separaten "aktuell nicht erreichbar" Status am Fahrzeug setzen.

Minsoc muss angewandt werden- sonst bleibt das Auto leer!

Hier müssen wir entweder diese Felder bei der Initialisierung überspringen oder (wäre mein Favorit) einen separaten "aktuell nicht erreichbar" Status am Fahrzeug setzen.

Ich denke mal laut. Z.B. ein "ich bin ein offline Fahrzeug aber nur weils grad dumm läuft" Feature? Das bräuchte dann kein neues Api.

@naltatis
Copy link
Member

Ich denke mal laut. Z.B. ein "ich bin ein offline Fahrzeug aber nur weils grad dumm läuft" Feature? Das bräuchte dann kein neues Api.

Ja genau. Wir brauchen aber irgendein Merkmal (nicht " Offline" Endung im Namen) an der wir das erkennen können und dann die Logiken und Darstellungen daran anpassen.

Weiteres Thema: Dieses Merkmal sollte sich auch im Betrieb ändern können. Heißt ein beim Start nicht erkanntes Fahrzeug (kein Empfang, ...), sollten wir in regelmäßigen Abständen wieder auf Verfügbarkeit checken und im Erfolgsfall den vollen Funktionsumfang wieder freischalten. Aktuell muss der User manuelle Neustarts machen, um zu prüfen, ob es geht. Das betrifft allerdings nicht nur Fahrzeuge, sondern all unsere Devices, Tarife, .... Ich weiß, dass das ein größerer technischer Umbau ist (Konzept Healthcheck). Aber das ist das, wo wir aus Nutzersicht hin müssen.
Vielleicht Out-of-Scope für dieses konkrete Problem hier, aber geht mir eher um die Richtung.

@StefanSchoof
Copy link
Contributor

Müsste es dann auch während evcc läuft auch in die andere Richtung gehen? Also von Online auf Offline Fahrzeug, weil der sich über längere Zeit nicht abrufbar ist.

@andig
Copy link
Member

andig commented Mar 10, 2024

@naltatis “wieder zum Leben erwecken“ wäre nett, und könnte man auf der Basis vllt. Machen- denn dann erkennen wir die halbtoten Fahrzeuge ja. Eins nach dem Anderen. Wie sollte das „Feature“ heissen? Sowas wie Repairable? Ich würde offline trotzdem dran lassen, denn für die meisten Zwecke ist das erstmal korrekt.

@naltatis
Copy link
Member

Wie sollte das „Feature“ heissen? Sowas wie Repairable?

Würde da eher aus einer Zustandssicht draufschauen: status: ok|unreachable, available: false, healthy: false, ...

@andig
Copy link
Member

andig commented Mar 10, 2024

So funktionieren die Features aber nicht. Wenn du das willst brauchts eine andere Lösung. Eins nach dem Anderen?

@andig andig added enhancement New feature or request and removed wontfix This will not be worked on labels Mar 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants