Reset vehicle cache when charging stops for last soc update

[andig]

Refs #7032 (comment)

Note: this will reset all vehicles; but that seems acceptable at this stage.

[premultiply]

Das betrifft dann aber nur die momentan verbundenen bzw. einem LP zugeordneten Fahrzeuge, oder?

[andig]

Alle. Genau genommen jegliche Verwendung von ResettableCached im Code.

[premultiply]

Mmmmh. Nicht dass dann im Zweifelsfall einmal der gesamte Fuhrpark erwacht...

[andig]

Das tut der ja eh falls es dauern gepollt wird (always) oder häufiger falls er verbunden ist (connected). Wenn beides nicht der Fall ist passiert auch nix.

Ist übrigens bei Fahrzeugerkennung identisch- da wird auch einmal der Cache geputzt um alle Fahrzeuge durchzuschauen falls innerhalb vehicleDetectInterval (1m) nix gefunden wurde. Anders gehts nicht...

[nename0]

Vielleicht müsste man dann jetzt das didChargeOnLastSocUpdate wieder raus nehmen, ansonsten könnte er zweimal updaten (einmal direkt nach dem Cache-Reset und dann nach dem Poll Interval nochmal).

[MarkusGH]

Vielleicht müsste man dann jetzt das didChargeOnLastSocUpdate wieder raus nehmen, ansonsten
könnte er zweimal updaten (einmal direkt nach dem Cache-Reset und dann nach dem Poll Interval nochmal).

Denke nicht - denn im Poll Mode "charging" würde sonst gar nicht mehr aktualisiert. Der Cache Reset löst meines Erachtens selbst keinen Poll aus sondern ermöglicht ihn nur.

[nename0]

Denke nicht - denn im Poll Mode "charging" würde sonst gar nicht mehr aktualisiert. Der Cache Reset löst meines Erachtens selbst keinen Poll aus sondern ermöglicht ihn nur.

Doch:

evcc/core/loadpoint_vehicle.go

Lines 337 to 340 in e235206

	// update if connected and soc unknown
	if lp.connected() && lp.socUpdated.IsZero() {
	return true
	}

[MarkusGH]

Stimmt!

[andig]

Ihr habt mich verloren. Bedeutet? Rein damit oder Veto?

[MarkusGH]

Meine Meinung : Rein damit - erwartetes Worst Case Szenario sind zwei Polls nach Abschluss des Ladevorgangs. Muss man beobachten und ggf. didChargeOnLastSocUpdate rausnehmen. Ist nicht mein PR, deswegen kann ich es nicht präventiv machen.

[MarkusGH]

Ausprobiert. Jetzt bekommt man im Poll Mode "charging" erwartungsgemäß ein Update wenn das Laden endet und ein weiteres wenn der Poll Timer abläuft. Fix: #7275