Skip to content
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

ABB: Fix implementation #3549

Merged
merged 22 commits into from
Jun 21, 2022
Merged

ABB: Fix implementation #3549

merged 22 commits into from
Jun 21, 2022

Conversation

premultiply
Copy link
Member

@premultiply premultiply commented May 30, 2022

  • Remove Phases1p3p() as it is not supported by command set (yet?)
  • Avoid modifying session status

@premultiply premultiply added the devices Specific device support label May 30, 2022
@premultiply
Copy link
Member Author

@andig Schön finde ich es noch nicht wirklich aber mir fehlt noch eine gute Idee. Sollten wir mal drüber quatschen.

@andig
Copy link
Member

andig commented Jun 4, 2022

Ne, das ist wirklich nicht schön. Müssen wir besprechen.

@andig
Copy link
Member

andig commented Jun 11, 2022

@premultiply @hopfi2k ich habs jetzt so geändert, dass status==5 als B gewertet wird. Das scheint erstmal richtig. Ebenso

enabled = status != 5 && strom > 0

Auch gut. Erstmal keine Modbus Fehler.

Für enable setze ich- falls status==5 einmalig "start" und dann den Strom. Ich komme auch korrekt wieder auf status=2, allein der Ladevorgang startet nicht.

Ich bin ratlos.

@andig andig self-assigned this Jun 11, 2022
@andig
Copy link
Member

andig commented Jun 11, 2022

PS.: ich sehe auch, dass der Charger immer mal wieder von alleine von status==2 auf status==5 zurück geht. Grund unklar. Teilweise ist dann auch

error 0x4000

zu sehen mit Verweis auf das Handbuch. Zu dem Fehler kann ich nichts finden.

@hopfi2k
Copy link

hopfi2k commented Jun 11, 2022

ModBus-Kommunikation abgebrochen (0x4000) Mögliche Ursache B- und/oder C-Phase fehlen. Es kam zu einem Abbruch bei der ModBus-Kommunikation.

@andig
Copy link
Member

andig commented Jun 11, 2022

Na dann stimmt auch irgendwas nicht an der Verkabelung? Oder er mag es einfach nicht wenn evcc sich verabschiedet.

@hopfi2k
Copy link

hopfi2k commented Jun 11, 2022

Na dann stimmt auch irgendwas nicht an der Verkabelung? Oder er mag es einfach nicht wenn evcc sich verabschiedet.

Die Verkabelung stimmt - da bin ich sicher!

@andig
Copy link
Member

andig commented Jun 11, 2022

Na dann stimmt auch irgendwas nicht an der Verkabelung? Oder er mag es einfach nicht wenn evcc sich verabschiedet.

Die Verkabelung stimmt - da bin ich sicher!

Glaub ich Dir, aber irgendwas muss den Fehler ja auslösen...

@hopfi2k
Copy link

hopfi2k commented Jun 11, 2022 via email

@premultiply
Copy link
Member Author

premultiply commented Jun 16, 2022

Ich glaube das Beste wäre hier das gesamte Sessionhandling soweit wie möglich der WB zu überlassen.

Entweder die Box ist immer freigegeben und startet somit automatisch eine Session sobald angesteckt wurde und beendet diese ggf. beim Abstecken oder sie wartet zunächst auf eine lokale RFID-Freigabe und startet dann die Session.
evcc greift nicht in die Session ein - wenn möglich.
Offensichtlich kann und muss man dies über die Konfigurationsapp entsprechend einrichten.

Eine inaktive Session wäre dann wie disconnected zu behandeln.

@premultiply
Copy link
Member Author

Na dann stimmt auch irgendwas nicht an der Verkabelung? Oder er mag es einfach nicht wenn evcc sich verabschiedet.

Ich gege davon aus das da auch ein Timeout bei der Modbus-Kommunikation wirkt. Das gibt es bei anderen Boxen auch.
Wenn da X Sekunden keine (definierte) Modbus-Kommunikation stattfindet wechselt die Box in einen Fehlerzustand.
Ist nur die Frage was genau die Box erwartet oder wann sie den Fehler triggert.

Dürfte dann aber im Normalbetrieb nicht vorkommen.

Vielleicht greift das hier auch nur da die Session noch über Modbus gestartet wurde.

@andig
Copy link
Member

andig commented Jun 16, 2022

Das gibt es bei anderen Boxen auch.

Da gibts egtl. immer irgendein Heartbeatregister. Das hab ich hier in der Doku nicht gefunden. Insofern Verhalten unklar :(

@premultiply
Copy link
Member Author

Bei Bender & Heidelberg geht das z. B. rein über regelmäßige Modbus-Kommunikation ohne spezielle Watchdog-Register.

@premultiply
Copy link
Member Author

@hopfi2k Du könntest auf deinem Pi mal mit ~/evcc/evcc -c ~/evcc/evcc.yaml --log trace testen ob und was genau geht.
Wichtig sind die Szenarien Modus-Umschaltung mit und ohne angeschlossenem Fahrzeug sowie mit und ohne "Kostenlosen Laden" und manueller Freischaltung via RFID oder App. Hier ist jeweils interessant was evcc bzgl. Ladestatus anzeigt oder welche Fehlermeldungen ggf. auftauchen.

evcc sollte ohne Freischaltung nun hoffentlich die Klappe halten. Das mit den gesperrten Registern ohne Session ist echt maximal dämlich in der Firmware implementiert. Welchen Sinn hat es dass man das Stromregister nur beschreiben darf wenn eine Session läuft??

@hopfi2k
Copy link

hopfi2k commented Jun 17, 2022 via email

@premultiply
Copy link
Member Author

Ich hab nochmal was gefixt dass ich erst jetzt mit ladendem Auto feststellen konnte. Nicht immer alles so selbstverständlich wie sich Boxen verhalten...

@hopfi2k Hab es dir nochmal auf deinem Pi gebaut. Einfach evcc nochmal auf der Konsole wie oben neustarten und bei Gelegenheit testen.

@hopfi2k
Copy link

hopfi2k commented Jun 17, 2022 via email

@hopfi2k
Copy link

hopfi2k commented Jun 18, 2022 via email

@premultiply premultiply marked this pull request as ready for review June 18, 2022 22:17
@premultiply premultiply added the bug Something isn't working label Jun 21, 2022
@premultiply premultiply merged commit 85335ed into master Jun 21, 2022
@premultiply premultiply deleted the fix/abb-comms branch June 21, 2022 14:31
@hopfi2k
Copy link

hopfi2k commented Jun 22, 2022

So ganz sauber läuft das leider immer noch nicht...gerade die aktuellste Master Branch kompiliert und probiert (am Tesla):

[lp-1  ] DEBUG 2022/06/22 19:49:16 charge currents: [4.92 4.96 4.99]A
[lp-1  ] DEBUG 2022/06/22 19:49:16 detected phases: 3p
[abb   ] DEBUG 2022/06/22 19:49:16 status: 4
[lp-1  ] DEBUG 2022/06/22 19:49:16 charger status: C
[abb   ] DEBUG 2022/06/22 19:49:16 status: 4
[lp-1  ] WARN 2022/06/22 19:49:16 charger out of sync: expected enabled, got disabled
[abb   ] DEBUG 2022/06/22 19:49:16 status: 4
[lp-1  ] WARN 2022/06/22 19:49:16 charger logic error: disabled but charging
[lp-1  ] DEBUG 2022/06/22 19:49:16 pv charge current: 0.377A = 6A + -5.62A (3880W @ 3p)

Am letzten WE mit dem ZOE war's noch wilder, da ging Laden überhaupt nicht. Muss ich aber die Tage nochmal genauer testen.

@premultply + @andig die nächsten 1-2 Stunden lass ich den Tesla dran hängen und wenn's eure Zeit erlaubt, könnt ihr ja nochmal schauen...

@premultiply
Copy link
Member Author

premultiply commented Jun 22, 2022

Ich doof. Ich glaube ich hab den Fehler schon gefunden.
Copy&Paste-Fehler beim minCurrent...

Wenn du nochmal ansteck könntest dann kann ich direkt checken.

@hopfi2k
Copy link

hopfi2k commented Jun 22, 2022 via email

@premultiply
Copy link
Member Author

Ich glaube du musst nochmal neu anstecken.
Wenn die Steuerung ausfällt kill die Box glaube ich die Ladesession, oder?
Jedenfalls sieht es für evcc aktuell so aus als wäre nichts angeschlossen.

@hopfi2k
Copy link

hopfi2k commented Jun 22, 2022 via email

@premultiply
Copy link
Member Author

premultiply commented Jun 22, 2022

Läuft. Dieser Fehler ist behoben.

Ich würde da aber gerade noch gerade etwas anders gucken bei der Gelegenheit:

Jetzt bitte nochmal ab und wieder anstecken und mir dann sagen was genau passiert.

Denn eigentlich sollte das Fahrzeug jetzt nicht auch nicht von alleine mit dem Laden beginnen nach dem neu anstecken.

@hopfi2k
Copy link

hopfi2k commented Jun 22, 2022 via email

@premultiply
Copy link
Member Author

premultiply commented Jun 22, 2022

Ok.

In Wahrheit fängt die Box tatsächlich kurz unerlaubt an zu laden weil sie nach dem Abstecken alles vergisst.
evcc bemerkt das und gibt dann (nur noch) eine charger out of sync meldung aus und dann wird sofort wieder gestoppt.
Vorher hat evcc leider keinerlei Möglichkeit etwas dagegen zu tun, da die Box ausserhalb einer Ladesession sämtliche Steuerbefehle unterbindet und die letzten vergessen hat.

Vielleicht kann man da noch irgendwas abfangen so dass die Meldung auch nicht mehr kommt. Ist aber erstmal nichts tragisches.

Fix f1284c1 ist im Master.

@hopfi2k
Copy link

hopfi2k commented Jun 22, 2022 via email

@hopfi2k
Copy link

hopfi2k commented Jun 23, 2022 via email

@premultiply
Copy link
Member Author

Falls du versucht hast die ZOE 3p wie den Tesla mit 6A zu laden (MinPV) ist das quasi erwartbar und hat nichts mit der Wallbox zu tun.

Probier mal ob das Problem auch auftritt wenn du minCurrent für die ZOE auf 8A (oder höher) stellst.

@hopfi2k
Copy link

hopfi2k commented Jun 24, 2022 via email

@premultiply
Copy link
Member Author

Erstmal um auszuschließen dass es daran liegt.

Hintergrund: Die ZOE (auch die neusten Modelle) ist bei 3p etwas empfindlich (vermutlichen wegen dem Netzfilter) und bricht dann gerne mal je nach örtlichen Bedingungen unvorhersehbar ab. 1p geht ohne Probleme auch mit 6A.

@premultiply
Copy link
Member Author

Hat es damit geklappt mit der ZOE?

@hopfi2k
Copy link

hopfi2k commented Jun 26, 2022 via email

dontbyte pushed a commit to dontbyte/evcc that referenced this pull request Aug 2, 2022
@hopfi2k
Copy link

hopfi2k commented Oct 11, 2022 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working devices Specific device support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants