-
-
Notifications
You must be signed in to change notification settings - Fork 636
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
Charger Delta AC MAX Smart (Modbus TCP) broken since v0.130.0 #15592
Comments
Here's the log I just made after downgrading to Log:
|
@premultiply What do you think? |
@Felsblick hast du die Version von gestern getestet? Dort wird der Wert ‚3‘ aus dem Register 1001 in den Status B übersetzt. |
@madmat17 ich habe es mit dem letzen release 0.130.4 getestet, dort wird Register 1000 ausgewertet und Wert 3 auf B gemappt, soweit richtig. Das funktioniert aber nur für die AC Max Smart. Bei der Basic muss Wert 3 auf Status A gemappt sein, da es dort bedeutet, das kein Fahrzeug eingesteckt ist wie ich es hier schon einmal geschrieben habe (woraufhin @premultiply das mapping geändert hatte, was jetzt bei der Smart zu Problemen geführt hat...). Daher der Lösungsvorschlag: Eigenes Template für die AC Max basic mit Nutzung des Registers 1001 mit dem Mapping aus evcc 0.128.4, was einwandfrei für die basic funktioniert hat. |
Das ist kein Template. Dann brauchen wir eine Weiche. Wie kann man Anhand der Register die Wallboxen zuverlässig unterscheiden? Bitte neues Issue dazu zur Übersicht. |
Das Register 1001 gibt es nur bei der Basic, die AC smart hat das nicht. Kann man das als Unterscheidungskriterium nutzen? |
Kommentar zwecks Übersichtlichkeit in @Felsblick 's neues Issue verschoben (#15697 (comment)) |
1 similar comment
Kommentar zwecks Übersichtlichkeit in @Felsblick 's neues Issue verschoben (#15697 (comment)) |
Describe the bug
Since v0.130.0 evcc no longer recognizes when a vehicle is connected to the Delta AC MAX Smart in a Modbus TCP configuration.
This is due to the change in the file
delta.go
, lines 154 & 156 (0.129.0...0.130.0#diff-6d9fb2ec3d3d401d69c4079b978d1c89d59bddfae6243ebeb45681a1ac125be5) where the EVSE state3
is falsely translated into charger status A - should be charger status B, though.Steps to reproduce
Configuration details
.
Log details
What type of operating system are you running?
Linux
Version
0.130.1
The text was updated successfully, but these errors were encountered: