-
-
Notifications
You must be signed in to change notification settings - Fork 640
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
Delta: fix Status & add StatusReason #15983
Conversation
Fragen wir mal andersrum: Es gibt von Delta ja eine vollständige Beschreibung der Register. Nur eine Version hält sich nicht dran. Dort gibt es Register 1001 (u.a.) einfach nicht. Welche Box ist die Ausnahme von der offiziellen Doku? |
Auch wenn man es so betrachtet, ist die Sache weder Schwarz noch weiß. Ich fasse mal zusammen: AC Max Basic:
AC Max Smart
Da wir nicht wissen, ob es noch weitere Firmwareversionen der boxen gibt oder in Zuklunft geben wird, die sich anders verhalten, ist mein Vorschlag für die Vermeidung von Fehler sich dort anzuhängen, wo die Antworten der Register jetzt schon Dokukonform ist und auch schon empirisch bewiesen wurde, dass evcc mit diesen Einstellung korrekt arbeitet.
|
Bitte mal testen. |
@Felsblick Ich darf dazu ergänzen: Falls jemand einen RS485 auf RJ45/TCP-Adapter als Leihe zur Verfügung stellen möchte, kann ich das gerne bei der Smart-Variante austesten. Ich werde mit der nächsten Nightly-Build einen Test fahren, ob die Erkennung für die Smart-Variante über Modbus TCP passt und berichten. |
Dann muss gemergt werden um es in den nigthly zu bekommen, oder? |
Es wäre super, nicht bei jedem einzelnen PR die gleiche Frage zu stellen. Danke :) |
Danke fürs Mergen und den neuen Release. Weiche funktioniert. Er springt auf Register 1001 bei der Basic. Leider ist der Wert 2 und 1 auf Register 1001 noch fehlerhaft auf Status A gemappt er muss aber Status B sein.
@madmat17 |
Gerne PR. Einfach die Datei hier auf Github wie gewünscht editieren. |
done |
@Felsblick |
Fixes #15963 #15963 (comment)