Circuits: review and validate logic from #20183

[andig]

See #20183, #19166 (comment):

Je länger ich drüber nachdenke, desto mehr frage ich mich ob hier nicht an der falschen Stelle eingegriffen wird. Warum sollte sich der Circuit darum scheren, ob etwas lädt? Müsste nicht stattdessen der Ladepunkt schon vorher sicherstellen, dass nur "echte" Anforderungen umgesetzt und an den Circuit geschickt werden? Ein bisschen analog #20286 wäre es vllt. besser den Algorithmus für Stromfreigabe anzupassen statt retrospektiv Korrekturen durchzuführen.

Weiterhin:

• fehlt eine Testabdeckung der neuen Logik
• hat Circuit: fix validateCurrent and validatePower #20183 das trace Log der Circuits entfernt

/cc @mfuchs1984 @premultiply

[andig]

@mfuchs1984 magst Du Dich mal bei uns auf Slack melden?

[mfuchs1984]

@andig @premultiply, mir kam gerade noch eine Erkenntnis, die ich für essentiell halte:

• ValidateCurrent bekommt als old lp.chargeCurrents übergeben, was dem vorher freigegeben Strom entspricht

  evcc/core/loadpoint.go

  Line 839 in 2460dbd

  currentLimit := lp.circuit.ValidateCurrent(lp.chargeCurrent, chargeCurrent, lp.charging())

• ValidatePower dagegen bekommt lp.chargePower, das der aktuellen gemessenen Ladeleistung entspricht, als old

  evcc/core/loadpoint.go

  Line 842 in 2460dbd

  powerLimit := lp.circuit.ValidatePower(lp.chargePower, currentToPower(chargeCurrent, activePhases), lp.charging())

-> schaut für mich nach einem Fehler in der Initialen Implementierung aus, ggf. durch die Bennennung von lp.chargeCurrent, diese lässt meinen es wäre der aktuelle Ladestrom, wahrend es eigentlich der freigegebe Ladestrom ist. Übergibt man überall die gemessenen Werte, also bei ValidateCurrent max(lp.chargeCurrents[0], lp.chargeCurrents[1], lp.chargeCurrents[2]) statt lp.chargeCurrent, dann entfällt der charging Parameter und die potential Berechnung ist korrekt, #19166 wäre so nie passiert.

Und hier mich ein paar lose Gedanken:

• bei ValidatePower ist der Aufruf korrekt. In Verbindung mit charger ohne Meter gibt's aber ein Problem mit der Reihenfolge der Funktionsaufrufe, die dafür sorgt, dass das load management nicht funktioniert, siehe Lastmanagement: charger flapping on and off - evaluates 0W->7360W twice #19473 (comment)
• möglicherweise gibt's einen Konflikt zwischen load management und loadpoint priority, wenn mehrere loadpoints im gleichen circuit sind? Wenn der erste den circuit auslastet, bekommt der zweite auch bei höherer Priorität nichts ab, wenn ich mich nicht irre, da ValidatePower und ValidateCurrent nichts freigeben.

[mfuchs1984]

Wenn ich nicht ganz falsch liege, könnte der ganze Aufwand umsonst gewesen sein und es hätte einfach nur der falsche Parameter korrigiert werden müssen 😒 😀

[premultiply]

In der Tat ist die Bezeichnung lp.chargeCurrent für die Stromvorgabe vs lp.chargeCurrents / lp.chargePower maximal unglücklich und verwirrend.
Da bekomme ich auch einen Knoten ins Hirn.

Da wäre lp.maxCurrent oder lp.offeredCurrent wohl eine passendere Bezeichnung...

[mfuchs1984]

lp.offeredCurrent oder lp.currentOffered wären für mich am intuitivsten.