forceVehicleUpdate to optimize vehicle detection

[thierolm]

Ich habe an meinem go-e loadpoint zwei Vehicle (Zoe, Audi E-Tron) definiert.
Dabei ist der/die/das Zoe :-) als erstes und der Audi als zweites Vehicle definiert.
Der cache Parameter sitzt für beide Vehicle auf 5m...

Wenn ich das Ladekabel vom Zoe auf den E-Tron umstecke, erkennt evcc das nicht korrekt und es werden im Frontend die alten Infos zum Zoe angezeigt.
Grund dafür sind m.E. die gecachten Vehicle Parameter, was ja ein Standard für alle Vehicle Module ist.
Wenn beim Anstecken des Audi in der lp.Update Funktion findVehicle ausgelöst wird, greift diese noch auf die gecachten Vehicle Werte zu.

Eine Idee zur Lösung wäre das Zurücksetzen der cache Parameter der beiden Vehicle.
Dazu wäre es meines Erachtens ausreichend einfach den Parameter Cached.updated (siehe provider/cache.go) für alle Vehicle am loadpoint auf c.clock.Now().Add(-c.cache) zu setzen, um eine Aktualisierung aller Vehicle Infos am Loadpoint zu erzwingen.

Das könnte man in der core/loadpoint.go durch eine neue Funktion forceVehicleUpdate erreichen, die man in der updateChargerStatus bei der disconnected Erkennung aufrufen kann ...

Ich habe in meinem Fork schon angefangen in dieser Richtung zu Coden, stehe aber vor der Challenge, wie ich aus der loadpoint.go am besten auf die Cache Parameter der Vehicle zugreifen kann. Die Vehicle API liefert mir nicht die cache Parameter ...

Freu mich auf alle Cents von euch ... :-)

[andig]

Verstehe ich Dich richtig: beim Umstecken werden die Fahrzeuge zwar abgefragt, aber weiter die alten Werte aus dem Cache gelesen? D.h. wir bräuchten eine (globale) Möglichkeit mal alle Caches (oder nur die der Fahrzeuge) zu leeren?

[thierolm]

Verstehe ich Dich richtig: beim Umstecken werden die Fahrzeuge zwar abgefragt, aber weiter die alten Werte aus dem Cache gelesen?

Ja, für das "Umsteck" Szenario sind m.E. eigentlich nur die Chaches der Vehicles, die für einen Loadpoint konfiguriert sind relevant. Alle Caches müsste man dann nicht zurücksetzen.
Wenn es allerdings simpler ist alle Chaches zurückzusetzen und dadurch keine "Kollateralschäden" zu erwarten sind, könnte man auch beim am Loadpoint Charger erkannten "disconnect" alle Chaches zurücksetzen.

[andig]

Schau mal ob das tut: #872

[thierolm]

Wow, das ging schnell, schaue ich mir an. Eine Frage, die Branch tickt noch auf der Struktur von Version 0.48. Ist das so von dir gewollt?

[andig]

Das ist sehr merkwürdig. Sollte sich trotzdem mergen lassen?

[thierolm]

Ich habe mir schnell deine Änderungen in eine lokale neue branch, die auf der aktuellen 0.51 basiert kopiert. Die Änderungen waren ja schnell im PR erkennbar. Darüber ist gerade ein make all erfolgreich durchgelaufen gelaufen.
Werde morgen nach Feierabend eine "Umstöpsel" Test starten.

Ein kleiner Reminder noch:
Im evcc-config Repo habe ich noch 2 PRs für dich, die auf den Merge warten (Niu vehicle und Tasmota Pseudo Charger)