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

Initial support for Algodue UEM meters on CP0 and CP1 and as EVU-Kit #1960

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

dj3mu
Copy link
Contributor

@dj3mu dj3mu commented Oct 19, 2024

This commit adds support for Algodue UEM meters at internal chargepoints.
Tested with a modified openWB2 and a UEM80-4D R (Modbus RTU version).

Modbus Address is set to 155 for CP0 and 156 for CP1. Please adjust if does not fit the openWB adressing concepts (which I don't know).

Support for UEM80-4D E as EVU-Kit is also included. But necessary UI changes are not yet part of this PR.

Copy link
Contributor

@benderl benderl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Danke für das neue Zähler-Modul.
Bitte noch ein paar Anpassungen vornehmen, dann können wir das so integrieren.

packages/modules/common/algodue.py Outdated Show resolved Hide resolved
packages/modules/common/algodue.py Show resolved Hide resolved
packages/modules/common/algodue.py Outdated Show resolved Hide resolved
packages/modules/internal_chargepoint_handler/clients.py Outdated Show resolved Hide resolved
@benderl
Copy link
Contributor

benderl commented Nov 15, 2024

Same registers should be valid for Ethernet/Modbus-TCP models (e.g. UEM80-4D E). But since, according to my reverse-engineering of the code, ability to configure EVU Kit to use it requires UI changes and is generally more complex, I was not yet able to integrate that. Additionally the problem is that EVU-Kit has hard-coded Modbus-TCP Port numberr of 8899 but Algodue meter's Modbus Port is the standard-port 502 and cannot be changed.

Maybe it can more easily be integrated with an openWB-Flex-Kit. Hints are welcome. But it could also be integrated in a second pull request.

The "EVU-Kit" always consists of a Modbus/TCP to RTU converter and one or more Modbus/RTU devices. So implementing the Modbus/TCP models as another option of the kit would break this sheme.

The Modbus/TCP models could be im plemented by a modified "carlo_gavazzi" device and counter component.
https://github.com/openWB/core/tree/master/packages/modules/devices/carlo_gavazzi/carlo_gavazzi

Integration into the settings ui should also be straight forward. Please let me know if you have questions.
https://github.com/openWB/openwb-ui-settings/tree/main/src/components/devices/carlo_gavazzi/carlo_gavazzi

@dj3mu
Copy link
Contributor Author

dj3mu commented Nov 15, 2024

I've accepted all suggestions and added the EVU-Kit code for Algodue as instructed above.

UI changes in openWB/openwb-ui-settings#582

Unfortunately I have never done and have absolutely no toolchain for Web-Development.
So I've no idea how to transpile and test-wise deploy UI to openWB. Hence I also can't really test the Modbus-TCP device driver yet.

I'd propose to handle the above PR for UI and merge it together with this PR.
I'll then, if not yet working out-of-the-box, create a new PR with the integration fixes.

@LKuemmel
Copy link
Contributor

waiting for feedback for reset functionality of the network connection

@LKuemmel LKuemmel added the question Further information is requested label Nov 18, 2024
@dj3mu dj3mu changed the title Initial support for Algodue UEM meters on CP0 and CP1 Initial support for Algodue UEM meters on CP0 and CP1 and as EVU-Kit Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants