andig
commented
9 months ago Closed davidgiga1993 closed 7 months ago
andig
commented
9 months ago andig
commented
9 months ago Related discussion in https://github.com/evcc-io/evcc/issues/11280#issuecomment-1872559005
davidgiga1993
commented
9 months ago The question is: Does this make sense? Naively I would assume (and I think I said that already in the past):
This way the min/max is always correctly guarded and you can still make changes without the need of logical prioritization of certain values
andig
commented
9 months ago Exactly- except for this one exception. Seems we'll need to revert it. Only we don't know how. The problem is:
@premultiply mir gehen die Ideen aus :( @naltatis Du warst mit dem 0A default fürs Minimum unglücklich- das wäre hier eine Lösung
davidgiga1993
commented
9 months ago Zu meinen Verständnis: Woher kommt der default? Könnte man nicht annehmen dass der Wert welcher von der UI geändert werden kann den 6A default hat und alle anderen 0? Dann wäre es weiterhin modifizierbar und 6A der Standard sofern der Nutzer es nicht ändert
docolli
commented
9 months ago Der loadpoint hat über die evcc.yaml ein mincurrent=2A, für das vehicle ist übers GUI ein mincurrent=4A definiert worden und evcc arbeitet nun mit 2,4A?
Wie kann das sein, wenn die Logik minCurrent: Use the max of loadpoint, charger and vehicle minCurrent implementiert ist?
Siehe https://github.com/evcc-io/evcc/pull/11311
davidgiga1993
commented
9 months ago Der loadpoint hat über die evcc.yaml ein mincurrent=2A, für das vehicle ist übers GUI ein mincurrent=4A definiert worden und evcc arbeitet nun mit 2,4A?
Wie kann das sein, wenn die Logik
minCurrent: Use the max of loadpoint, charger and vehicle minCurrentimplementiert ist? Siehe #11311
Der Charger meldet die 2,4A, im Code gibt es ja den Sonderfall dass der min Wert vom Charger genommen wird sofern einer existiert
docolli
commented
9 months ago Achso, verstehe jetzt woher dieser "krumme" Wert kommt.
Aber selbst dann sollte die Logik gelten Use the max of loadpoint, charger and vehicle minCurrent.
Das sollten dann (auch wenn der charger 2,4A vorgibt) eben die von Dir am vehicle gewählten 4A sein, oder?
Der charger bzw. das Auto können ja ruhig melden, dass ein Minimalstrom von 2,4A möglich ist. Aber wenn der evcc Anwender hier gerne minimal 4A hätte, dann ist dies eben die Grenze und unterhalb von 4A wird nicht mehr geladen (auch wenn das vom Auto/Charger her möglich wäre).
VolkerK62
commented
9 months ago we need a default minimum (6A)
im Code gibt es ja den Sonderfall dass der min Wert vom Charger genommen wird sofern einer existiert
Ich vermute mal, nicht jeden Charger liefert einen Min-Wert? Wäre es möglich für Charger, die keinen liefern, diesen auf 6A zu setzen? Dann hätte man einen default und für die anderen, gilt der gelieferte Wert. Dann könnte man doch die "max()" Regel ohne Ausnahmen umsetzen.
docolli
commented
9 months ago Ich vermute mal, nicht jeden Charger liefert einen Min-Wert?
Ja, meiner hier ist so einer. Darum musste ich auch in der evcc.yaml beim loadpoint mincurrent: 6 definieren.
Man könnte vermutlich im Code prüfen ob am loadpoint ein mincurrent definiert ist, wenn nicht, dann könnte man 6A als default annehmen, ABER das würde mit EEBUS Wallboxen kollidieren.
Entweder diese müssen mit mincurrent: 0 definiert werden (um die 6A default auszuhebeln), oder (was logischer wäre) man führt eine config Syntax wie mincurrent: auto ein, um festzulegen, dass der loadpoint das mincurrent selber liefert.
Dann könnte man doch die "max()" Regel ohne Ausnahmen umsetzen.
Ja, das würde dann wieder intern so gehen, um das effectivemincurrent festzulegen.
davidgiga1993
commented
9 months ago Ich hab mal ein Draft erzeugt. Im MR ist ein Flussdiagram wie das minCurrent berechnet wird, tests sind auch angepasst. Feedback willkommen.
Update: Hab den Change bei mir ausgerollt und zumindest soweit ich das testen kann funktioniert alles
andig
commented
9 months ago @naltatis müssen wir durchsprechen.
DerAndereAndi
commented
9 months ago Der Min-Wert vom Charger ist entweder durch das verwendete Protokol IEC61851 definiert, dort ist er dann immer 6A. Bei Wallboxen welche über ISO kommunizieren können und ein Auto angeschlossen haben das über die AC Verbindung auch ISO unterstützt, wird der min-Wert aus dieser Kommunikation ermittelt. Und wenn bei diesen Wallboxen ein EV mit IEC61851 kommunizert, ist er 6A.
hidminds
commented
9 months ago Siehe auch #7241 .
hidminds
commented
8 months ago Wenn der minCurrent Wert vom Loadpoint genommen würde, wäre das für mich vermutlich ein Workaround dafür, dass der am vehicle eingestellte minCurrent derzeit auch nicht funktioniert.
Das sieht dann übrigens so aus.. Gesamt Out ist bei mir höher als IN. Das wäre schön! ;-)
Die Werte beim "Ausgang" sind falsch berechnet. Tatsächlich gelangt wirklich nichts ins Fahrzeug weil der Zoe mindestens 3p mit 8A geladen werden muss, und der Hausverbrauch ist in etwa bei 0,8 kWh.
TAEmty
commented
8 months ago Hi zusammen, ich kann das von hidminds gezeigte Verhalten reproduzieren, bei mir macht es allerdings das EVCC aufgrund der äußeren Umstände komplett unbrauchbar: Der Zoe meiner Frau fängt bei 8A an zu laden, seit etwa 0.123.0 (hat ne weile gedauert, bis ich raushatte, was los war, deshalb bin ich unsicher) bekomme ich aber in einem Modus < "schnell" immer nur 6A Ladestromvorgabe, was dazu führt, dass der Zoe gar nicht lädt. Ich habe eine Elli Wallbox über EEBUS am EVCC.
edit: Ok, das ist EXAKT, was er geschrieben hat. Ich kanns also 1:1 reproduzieren.
hidminds
commented
8 months ago Hi @TAEmty !
edit: Ok, das ist EXAKT, was er geschrieben hat. Ich kanns also 1:1 reproduzieren.
Ja richtig, wir haben denke ich exakt dasselbe Problem. Ich habe auch einen EEBus Elli Charger und vor 0.123 hatte ich die minimale Stromstärke auf 8A am Zoe vehicle einstellen können und das hatte auch funktioniert. Seit 0.123 ist nun leider mit evcc nur schnelles Laden ( bei mir mit 11kW Leistung ) oder batterieunterstütztes Laden bei Sonnenschein möglich wenn evcc mal über 8A/5,5kW am Charger einstellt. Damit beim Renault Zoe überhaupt Strom in die Autobatterie gelangt, benötigt man leider eine effektive minimale Stromstärke Einstellung von 8A bei 3 Phasen = 5,5 kW Leistung. Vor 0.123 hatte die mincurrent Einstellung am vehicle noch funktioniert. Jetzt wird am Charger warum auch immer die Stromstärke unter 8A eingestellt/geregelt. Das ist per api Aufruf auch ersichtlich. Unter 5,5kW steht in der UI, dass der laden würde und es wird dann auch bereits der neue Akku Füllstand erhöht aber tatsächlich geht unter 8A/5,5kW gar nichts in den Akku von diesem speziellen EAuto.
davidgiga1993
commented
8 months ago Nur als kleine Anmerkung: Das Problem wurde in meinem MR bereits behoben. Wer will kann mein fork bauen, da drin geht alles. Kann die Tage auch noch ein Release veröffentlichen
hidminds
commented
8 months ago Nur als kleine Anmerkung: Das Problem wurde in meinem MR bereits behoben. Wer will kann mein fork bauen, da drin geht alles. Kann die Tage auch noch ein Release veröffentlichen
@davidgiga1993 danke, deine Changes würde ich gerne mal testen. Hab evcc noch nie gebaut. Könnte/Sollte das aber mal machen. Dann könnte ich evtl selbst auch mal forken und PRs zur Diskussion erstellen... ;-)
Ich habe mir deine Änderungen angeschaut. Ich denke, dass es für mein Problem idealer wäre, wenn ich wieder am vehicle einen absoluten mincurrent einstellen könnte. D.h. nur wenn die minCurrent Einstellung am vehicle gesetzt ist, und dieses an einem Loadpoint aktiv ist, soll effektiv der Loadpoint minCurrent auf das am vehicle definierte minCurrent eingestellt werden. Der current am Loadpoint darf dann nie darunter geregelt werden.
Ich werde bspw ab Sommer noch ein zweites EAuto (Tesla) dort öfter mal laden. Wenn der Tesla an dem Loadpoint angesteckt ist, soll der Elli Charger die Lade Stromstärke bitte NICHT auf 8A Minimum einstellen. Bei dem Tesla sollte das Laden bereits ab 6A funktionieren. Beim Zoe soll evcc wieder das Minimum auf 8A einstellen. Eben je nach erkanntem vehicle.
Ramada2020
commented
8 months ago Hallo. Gibt ein schon ein Idee wann dieser Fix in das offizielle Release eingebaut wird. Habe mit meinem Twingo das gleiche Problem. Danke. #12770
TAEmty
commented
7 months ago Nur als kleine Anmerkung: Das Problem wurde in meinem MR bereits behoben. Wer will kann mein fork bauen, da drin geht alles. Kann die Tage auch noch ein Release veröffentlichen
Hi davidgiga1993. Danke für die Info und die Mühe!. Ich bin unsicher, ob ich das von meiner Qualifikation her hinbekomme und dann auch irgendwann den Weg zurück. Ich warte, wenn ich auf mich allein gestellt bin, lieber auf ein release. Aber: EEBUS und die Elli Wallbox ist ne ziemliche Katastrophe, das hab ich mittlerweile mitbekommen. Buggy und instabil, zum Teil dämlich gelöst. Wenn es der Allgemeinheit hilft, schick ich gerne Logs, mach Versuche mit ID.4 <-> und Zoe an der Elli. Wenn ich mit was konretem helfen kann, werde ich das gerne machen.
andig
commented
7 months ago Dieser Fehler ist schon viel zu lange offen. Siehe https://github.com/evcc-io/evcc/pull/12866 für einen Vorschlag:
GetMinMaxCurrent()) unterstützt.Ein gemischter Betrieb Tesla+IEC Fahrzeug (VW) am TWC3 wobei der Tesla dann 5A Minimum hätte ist damit noch nicht möglich. hier aber unkritisch da Ladesteuerung bisher ausschließlich für Tesla umgesetzt ist.
andig
commented
7 months ago Habt ihrs mal getestet?
hidminds
commented
7 months ago @andig Deine Änderungen im commit sehen auf den ersten Blick schon mal vielversprechend aus. Vielen Dank! :+1: Ich habe mir gerade den aktuellen master gepullt, cross compiled und auf meinem raspi eingespielt. Mein Zoe ist aktuell zu 100% geladen und es ist dunkel. Ist also gerade schlecht mit testen im Real-World-Szenario :wink: . Ich werde deinen Fix die nächsten Tage mit allen Lademodi mal testen und werde Bescheid geben.
hidminds
commented
7 months ago @andig Habs heute Nachmittag mal testen können.
Ich hatte mit einer "sauberen" (ohne irgendwelche lokale Änderungen) 0.124.9~next Version von vorgestern mit deinen Änderungen getestet.
Funktioniert wunderbar. Test mit meinem Renault Zoe an meiner Elli Charger Pro Wallbox bei etwas Sonnenschein. Es folgen JSON api/state Abzüge vom Test:
1-pv-0.124.9-next.json 2-minpv-0.124.9-next.json 3-now-0.124.9-next.json 4-pv-0.124.9-next.json
Der Zoe wurde durchgängig in allen Modes effektiv mit mindestens ~5,5 kW geladen. Die effective MinCurrent wurde wie am vehicle definiert immer auf minimal 8A eingestellt.
Herzlichen Dank! 🍻
andig
commented
7 months ago Na gottseidank 😂. Danke fürs Feedback und sorry, dass das so lange gedauert hat. Da hab ich den Wals vor lauter Bäumen nicht gesehen…
andig
commented
4 months ago @davidgiga1993 hatte der Fix für Dich den Zweck erreicht? https://github.com/evcc-io/evcc/issues/14418 sagt mir, dass nicht alle Fälle abgedeckt sind :(
Describe the bug
When using the
Min+PVcharge mode, the minimum current set the by UI seems to be ignored.Steps to reproduce
Configuration details
Log details
What type of operating system are you running?
Linux
Version
0.123.7
Update
I just tested the
Schnellmode and it correctly uses themax currentfrom the UI: