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

Fix OpenWB 2.0 enabled state handling #11543

Merged
merged 2 commits into from
Jan 7, 2024
Merged

Conversation

MarkusGH
Copy link
Contributor

@MarkusGH MarkusGH commented Jan 7, 2024

Fix #11517

Enabled state handling was not implemented correctly for OpenWB 2.0.
Fixed (same handling as in OpenWB 1.x, as OpenWB 2.0 still lacks a "real" enabled state)

Fix enabled state handling
Fix formatting
@andig
Copy link
Member

andig commented Jan 7, 2024

@MarkusGH ich werde aus dem Log nicht schlau. Kannst Du bitte mal prüfen, was die OpenWB für openwbRegActualAmps tatsächlich liefert, nachdem sie mit 0A abgeschaltet wurde? Ist nur der Vergleich mit 1 falsch oder kann man aus dem Register gar nichts ablesen?

/cc @snaptec

@MarkusGH
Copy link
Contributor Author

MarkusGH commented Jan 7, 2024

@MarkusGH ich werde aus dem Log nicht schlau. Kannst Du bitte mal prüfen, was die OpenWB für openwbRegActualAmps tatsächlich liefert, nachdem sie mit 0A abgeschaltet wurde? Ist nur der Vergleich mit 1 falsch oder kann man aus dem Register gar nichts ablesen?

@andig: Ich habe hier keine OpenWB mit 2.0 Software. Aber ein Fehler ist aus meiner Sicht offensichtlich - die OpenWB kennt auch in der Software 2.0 keine Unterscheidung zwischen enabled und disabled, also muss man es wohl in Software emulieren, so wie bei anderen Wallboxen auch.

@andig
Copy link
Member

andig commented Jan 7, 2024

Das macht nur wenig Sinn weil das ja über das Stromregister gesteuert wird. Dann frag ich beim OP mal nach.

@andig andig marked this pull request as draft January 7, 2024 14:47
@MarkusGH
Copy link
Contributor Author

MarkusGH commented Jan 7, 2024

@andig: Mach wie Du willst - aber wenn Du Dir Aufwand sparen möchtest, dann schau Dir doch einfach mal die funktionierende OpenWB 1.x Anbindung an. Denn wenn man den enabled state nicht speichert, kriegt man das Problem, dass sich die Wallbox selbständig "disabled" wenn das Fahrzeug nicht lädt (z.B. weil die Batterie voll ist) oder. Der Fehler "charger out of sync: expected enabled, got disabled" sollte damit mindestens weg sein. Und der andere wird auch verschwinden, da er wohl daraus resultiert, dass der Charger State "C" meldet, aber der gemeldete Ladestrom noch 0 ist.

@andig
Copy link
Member

andig commented Jan 7, 2024

Ah, jetzt machts klick. Danke!

@andig andig marked this pull request as ready for review January 7, 2024 14:58
@andig andig merged commit a55ce94 into evcc-io:master Jan 7, 2024
6 checks passed
@MarkusGH MarkusGH deleted the patch-4 branch January 8, 2024 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OpenWB Software 2.0: Warning for incorrect charger status
2 participants