-
-
Notifications
You must be signed in to change notification settings - Fork 684
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
EEbus: cannot configure higher minCurrent than charger claims #11636
Comments
That's intentional: evcc/core/loadpoint_effective.go Lines 62 to 83 in da966f5
|
Related discussion in #11280 (comment) |
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 |
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 :( |
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 |
Der loadpoint hat über die evcc.yaml ein mincurrent=2A, Wie kann das sein, wenn die Logik |
Der Charger meldet die 2,4A, im Code gibt es ja den Sonderfall dass der min Wert vom Charger genommen wird sofern einer existiert |
Achso, verstehe jetzt woher dieser "krumme" Wert kommt. 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). |
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
Ja, das würde dann wieder intern so gehen, um das |
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 |
@naltatis müssen wir durchsprechen. |
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. |
Siehe auch #7241 . |
Hi zusammen, edit: Ok, das ist EXAKT, was er geschrieben hat. Ich kanns also 1:1 reproduzieren. |
Hi @TAEmty !
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. |
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 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. |
Hallo. Gibt ein schon ein Idee wann dieser Fix in das offizielle Release eingebaut wird. Habe mit meinem Twingo das gleiche Problem. Danke. #12770 |
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. |
Dieser Fehler ist schon viel zu lange offen. Siehe #12866 für einen Vorschlag:
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. |
Habt ihrs mal getestet? |
@andig Deine Änderungen im commit sehen auf den ersten Blick schon mal vielversprechend aus. Vielen Dank! 👍 |
@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.
1-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! 🍻 |
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… |
@davidgiga1993 hatte der Fix für Dich den Zweck erreicht? #14418 sagt mir, dass nicht alle Fälle abgedeckt sind :( |
Describe the bug
When using the
Min+PV
charge 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
Schnell
mode and it correctly uses themax current
from the UI:The text was updated successfully, but these errors were encountered: