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

Enercab Smart ( ELEDIO - EVCC01 ) basic support #13990

Merged
merged 8 commits into from
Jun 2, 2024

Conversation

franzongit
Copy link
Contributor

Enercab Smart ( ELEDIO - EVCC01 https://www.enercab.at/starkstrom-11kw-3x16a/1219-enercab-smart-3x16a-400cee-11kw-100m-starkstrom-intelligentes-ladekabel-9120099491495.html ) can be used in OCPP mode, however when switching to websocket it sends "file://" as source string which by default is rejected by lorenzodoninis library, so a handler has to be set to accept it.
The next problem is that the power value is sent as a space padded string which has to be trimmed.

If EVCC is killed during a running transaction and the car is unplugged or requests a stop, the box tries to get a StopTransaction message through even after being power cycled - this prevents the initialization of a new EVCC instance. So, if such a StopTransaction is received before the new connection is actually initialized, it is rejected by an additional patch.

I'm still trying to figure out how to reliably switch phases, I ran into the StopTransaction issue using #12861 and I have not included the reset code for easier integrability - not even a hard reset gets that resending of StopTransaction away, after initially believing that I had bricked the box I found out that iobroker-ocpp was able to revive it, then I realized that the Transaction was always repeated.

@andig andig added enhancement New feature or request devices Specific device support labels May 21, 2024
charger/ocpp/cp_core.go Outdated Show resolved Hide resolved
charger/ocpp/cp_core.go Outdated Show resolved Hide resolved
@andig andig marked this pull request as draft May 26, 2024 15:07
@andig andig marked this pull request as ready for review May 27, 2024 09:01
@andig
Copy link
Member

andig commented May 27, 2024

Mhhhm, das mögen jetzt die OCPP Tests nicht, muss ich nochmal rein schauen.

@franzongit
Copy link
Contributor Author

Vorher war dort auch ein Error in den tests, aber wenn ich mich dachte war der weil ich eine ERROR message ausgegeben hatte aber keine retourniert wurde. Jetzt erwartet er vermutlch einen error bei einem conn==nil testcase, der in diesem Fall aber gerade nicht auftreten soll.

@andig
Copy link
Member

andig commented Jun 2, 2024

Vorher war dort auch ein Error in den tests, aber wenn ich mich dachte war der weil ich eine ERROR message ausgegeben hatte aber keine retourniert wurde. Jetzt erwartet er vermutlch einen error bei einem conn==nil testcase, der in diesem Fall aber gerade nicht auftreten soll.

Mhhm- ich sehe im Master keine Fehler?

@andig andig merged commit 68cc29a into evcc-io:master Jun 2, 2024
6 checks passed
@franzongit
Copy link
Contributor Author

danke! ich versuche mal die phasen hinzubekommen, mir scheint es, als ob der phasenwechsel nur bei einem transaktionsende funktioniert, was dann mit setChargingProfile 0a nicht zusammengeht

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devices Specific device support enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants