Added Modbus to OpenDTU #582

ArekKubacki · 2023-02-13T10:03:32Z

Added Modbus TCP/IP

ArekKubacki · 2023-02-27T13:31:18Z

Hello,
Any idea why there is error? In VisualStudio Code there is not any error.

PhiDiWoe · 2023-03-04T21:08:20Z

Ist die Modbus unterstützung dann auch für das OLIMEX Modul gedacht? also man kann dann direkt per LAN damit arbeiten?

horfic · 2023-03-08T15:29:05Z

the build fails as it can't find the "ModbusDTU.h" file you include

mzurhorst · 2023-05-01T15:24:48Z

For my understanding: will this PR provide the capability to read the Hoymiles power/energy from e.g. a Kostal Smart Energy Meter? -- Or is there another use case behind?

This reverts commit 5617240ce1fb15a3796d1ed558cc89f9ba72d8d5.

ArekKubacki · 2023-06-05T09:52:57Z

This change allows you to read data from OpenDTU over Modbus. It runs parallel to MQTT.

AloisKlingler · 2023-06-07T05:17:55Z

@ArekKubacki can you please offer me a compiled binary? I would like to try out if this can be added at fronius gen24 as "Power Meter"

ArekKubacki · 2023-06-07T05:47:52Z

firmware.zip

If you need Modbus registers, see here

https://github.com/ArekKubacki/OpenDTU/blob/master/src/ModbusDtu.cpp

AloisKlingler · 2023-06-07T07:56:07Z

to get sure: This would act as Modbus TCP (port 502 on OpenDTU-IP) or as Modbus RTU ?
is there additional config necessary?
I tried to read registers 40072 to 40193 - I think this would be sunspec, but not sure at all - but nothing there. the registers you mentioned are maybe (decimal) 4096 to 4115 ?

thanks. :-)

AloisKlingler · 2023-06-09T15:11:31Z

have it. this does the trick ... (also inverter totals used) :-)
float CurrPWR = (Datastore.getTotalAcPowerEnabled()*-1); uint16_t *myhex = (uint16_t *)&CurrPWR; mb.Hreg(0x9ca1, myhex[1]); mb.Hreg(0x9ca2, myhex[0]); float TotYLD = (Datastore.getTotalAcYieldTotalEnabled()*1000); uint16_t *myhex2 = (uint16_t *)&TotYLD; mb.Hreg(0x9cc1, myhex2[1]); mb.Hreg(0x9cc2, myhex2[0]);

AloisKlingler · 2023-06-09T15:23:28Z

success 🥳

AloisKlingler · 2023-06-21T09:25:25Z

is there a reason for not merging this PR into OpenDTU?

MIC1981 · 2023-06-30T19:34:07Z

firmware.zip

If you need Modbus registers, see here

https://github.com/ArekKubacki/OpenDTU/blob/master/src/ModbusDtu.cpp

I tried your precompiled firmware to use Modbus with iobroker. But everything i tried failed. If I try to read a register, for example 4096 I only get an error "illegal data address".
Is Modbus provided in the firmware you posted above? If not, is there a ready-to-flash firmware available?

fantilator · 2023-07-07T20:53:57Z

Any chance that this will be implemented in the master branch?

madmartin · 2023-07-18T07:28:43Z

@AloisKlingler @fantilator @ArekKubacki

is there a reason for not merging this PR into OpenDTU?

IMHO yes.

The commits mess around with libraries (I assume a change in CR/LF cause by an editor).
Every new feature which is not essentially necessary should be able to be turned on/off. At least at runtime, and if it consume a lot of codespace then it should be configurable at compile time also.
Pull requests should look "nice" in the history of the main project when they get merged. So if this would be my project, I would not merge a PR with 22 commits and half of it are "fix this" "fix that" "merge...". I recommend a rebase of the pull request.

This is just my personal opinion, I am not the project owner.

mschimpfhuber · 2023-08-18T06:00:56Z

Hello, I am new here,

@AloisKlingler
is it possible that I get the AC_Power in an Register?

heppth · 2023-10-30T10:33:52Z

Unfortunately, only the DC channels are provided. I would have been more interested in the AC channel.

stefan123t · 2023-10-30T10:40:27Z

@ArekKubacki have you considered / asked to add your PR/changes to the OpenDTU-onBattery fork from Helge Erbe ?

I know Helge has included a couple of plugins to retrieve Power Meter readings via MQTT, so maybe adding ModBus support there might be easier and straight forward ?

https://github.com/helgeerbe/OpenDTU-OnBattery

ArekKubacki · 2023-11-06T07:41:01Z

I think that one day I will come back to mastering Modbus and creating a subpage where I can set everything up. How closely does this version follow the original OpenDTU?

stefan123t · 2023-11-06T20:20:41Z

@ArekKubacki
I have not checked what your PR actually does, but OpenDTU-onBattery has a PowerMeter.cpp class which has support for several PowerMeter Sources, including HTTP, MQTT, SML and something called SOURCE_SDM1PH/3PH.
As far as I know the SDM Smart Meters are read via ModBus / RS485 too ?

https://github.com/helgeerbe/OpenDTU-OnBattery/blob/development/src/PowerMeter.cpp

Regarding the fork Helge seems to be rather quick in merging changes which are inteoduced by Thomas in upstream OpenDTU. I can not remember any delays in the past year. So thanks to the great jobs of both project maintainers this fork is really well maintained !

ArekKubacki · 2023-11-09T20:41:59Z

I'll look into it in my spare time.

pneutek · 2023-11-19T18:56:33Z

Hello

I'm new here. I am planning a PV project. Can I query the open DTU data (total power, power of the individual solar panels) with Modbus TCP? Is Modbus integrated in the current open DTU v23.11.16?
If so, is there a description here?

hwinkel · 2023-12-17T20:25:58Z

Could OpenDTU Emulate a Fronius MODBUS Interface and becomes compatible with a Victron ESS system?

AloisKlingler · 2023-12-18T05:33:11Z

Could OpenDTU Emulate a Fronius MODBUS Interface and becomes compatible with a Victron ESS system?

this does this fork here: https://github.com/AloisKlingler/OpenDTU-FroniusSM-MB

stefan123t · 2023-12-18T07:30:23Z

@AloisKlingler did you check the fork / code in https://github.com/helgeerbe/OpenDTU-OnBattery ?
I do recall that @helgeerbe already implemented a couple of things to work with Victron ESS.
If your code does something different, maybe it would be a good idea to merge it into helge's fork, as this is a perfectly well maintained OpenDTU fork with the same focus, i.e. ESS.

AloisKlingler · 2023-12-18T07:43:50Z

@AloisKlingler did you check the fork / code in https://github.com/helgeerbe/OpenDTU-OnBattery ? I do recall that @helgeerbe already implemented a couple of things to work with Victron ESS. If your code does something different, maybe it would be a good idea to merge it into helge's fork, as this is a perfectly well maintained OpenDTU fork with the same focus, i.e. ESS.

regrettably I can not - as I just forked @ArekKubacki 's implementation and played around until the Fronius Smartmeter Simulation worked

so more or less, my fork is stuck in the current state as I am not good enough in coding to provide a PR for OpenDTU-OnBattery

hwinkel · 2023-12-18T08:03:09Z

The point for me is not the meter emulation, metering is kind of easy, but the zero feed in control of the hoymiles. If the look like sunspec fronius to the Victron system they should be reduced in power if Victron says so?!?

OpenDTU is extended by a Modbus server. The Modbus server serves TCP at port 502. At Modbus ID 1 the server mimicks the Modbus registers in the original DTUPro. At Modbus ID 125 the server serves a SunSpec compatible pseudo inverter that provides the OpenDTU aggregated data from all registered inverters. The OpenDTU Modbus sources were imspired by : https://github.com/ArekKubacki/OpenDTU. See tbnobody#582 for the orignal pull request. The Modbus library used for Modbus communication is: https://github.com/eModbus/eModbus. Documentation for the library is here: https://emodbus.github.io/. The library was choosen over full featured libraries to achieve a lower memory footprint. fixes tbnobody#582 Signed-off-by: Bobby Noelte <b0661n0e17e@gmail.com>

OpenDTU is extended by a Modbus server. The Modbus server serves TCP at port 502. At Modbus ID 1 the server mimicks the Modbus registers in the original DTUPro. At Modbus ID 125 the server serves a SunSpec compatible pseudo inverter that provides the OpenDTU aggregated data from all registered inverters. The OpenDTU Modbus sources were imspired by : https://github.com/ArekKubacki/OpenDTU. See tbnobody#582 for the orignal pull request. The Modbus library used for Modbus communication is: https://github.com/eModbus/eModbus. Documentation for the library is here: https://emodbus.github.io/. The library was choosen to achieve a lower memory footprint. fixes tbnobody#582 Signed-off-by: Bobby Noelte <b0661n0e17e@gmail.com>

OpenDTU is extended by a Modbus server. The Modbus server serves TCP at port 502. At Modbus ID 1 the server mimicks the Modbus registers in the original DTUPro. At Modbus ID 125 the server serves a SunSpec compatible pseudo inverter that provides the OpenDTU aggregated data from all registered inverters. At Modbus ID 243 the server serves a Sunspec meter that provides aggregated AC power and AC yield values of all registered inverters. The OpenDTU Modbus sources were imspired by : https://github.com/ArekKubacki/OpenDTU. See tbnobody#582 for the orignal pull request. The Modbus library used for Modbus communication is: https://github.com/eModbus/eModbus. Documentation for the library is here: https://emodbus.github.io/. The library was choosen to achieve a lower memory footprint. fixes tbnobody#582 Signed-off-by: Bobby Noelte <b0661n0e17e@gmail.com>

OpenDTU is extended by a Modbus server. The Modbus server serves TCP at port 502. At Modbus ID 1 the server mimicks the Modbus registers in the original DTUPro. At Modbus ID 125 the server serves a SunSpec compatible pseudo inverter that provides the OpenDTU aggregated data from all registered inverters. At Modbus ID 243 the server serves a Sunspec meter that provides aggregated AC power and AC yield values of all registered inverters. The OpenDTU Modbus sources were imspired by : https://github.com/ArekKubacki/OpenDTU. See tbnobody#582 for the orignal pull request. The Modbus library used for Modbus communication is: https://github.com/eModbus/eModbus. Documentation for the library is here: https://emodbus.github.io/. The library was choosen to achieve a lower memory footprint. Signed-off-by: Bobby Noelte <b0661n0e17e@gmail.com>

ArekKubacki added 6 commits February 13, 2023 10:42

Add Modbus

4084b4d

Added Modbus

cc98831

Merge remote-tracking branch 'origin/master'

ddb0325

Checking that the measurement does not exceed the max Power

802601a

Modbus BugFix

0777a9c

Modbus BugFix

53b9690

ArekKubacki added 3 commits March 8, 2023 16:46

Added WatchDog to Modbus

e13348c

Update from master

32b9750

Merge branch 'master' into master

3277e71

ArekKubacki added 13 commits May 30, 2023 13:01

Revert "Merge branch 'master' into v1"

1b8b56f

This reverts commit 5617240ce1fb15a3796d1ed558cc89f9ba72d8d5.

Merge branch 'tbnobody-master'

314b285

Edit platformio.ini

6ec026b

Fixed Temperature, Frequency etc.

5077d03

Merge remote-tracking branch 'origin/master'

8d52334

Modbus fixed

a3b2457

Fixed Temp, Freq, AC

54e1641

Merge remote-tracking branch 'origin/master'

2d9c942

Merge

45c9984

Merge

adeed86

Modbus Added

0fb144f

ModbusDtu h file fixed

6765ab6

Fixed Modbus Library

f7678a0

b0661 mentioned this pull request Apr 8, 2024

Add Modbus #1893

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Added Modbus to OpenDTU #582

Added Modbus to OpenDTU #582

ArekKubacki commented Feb 13, 2023

ArekKubacki commented Feb 27, 2023 •

edited

PhiDiWoe commented Mar 4, 2023

horfic commented Mar 8, 2023 •

edited

mzurhorst commented May 1, 2023

ArekKubacki commented Jun 5, 2023

AloisKlingler commented Jun 7, 2023

ArekKubacki commented Jun 7, 2023 •

edited

AloisKlingler commented Jun 7, 2023 •

edited

AloisKlingler commented Jun 9, 2023

AloisKlingler commented Jun 9, 2023

AloisKlingler commented Jun 21, 2023

MIC1981 commented Jun 30, 2023

fantilator commented Jul 7, 2023

madmartin commented Jul 18, 2023 •

edited

mschimpfhuber commented Aug 18, 2023

heppth commented Oct 30, 2023

stefan123t commented Oct 30, 2023

ArekKubacki commented Nov 6, 2023

stefan123t commented Nov 6, 2023

ArekKubacki commented Nov 9, 2023

pneutek commented Nov 19, 2023

hwinkel commented Dec 17, 2023

AloisKlingler commented Dec 18, 2023

stefan123t commented Dec 18, 2023

AloisKlingler commented Dec 18, 2023

hwinkel commented Dec 18, 2023

Added Modbus to OpenDTU #582

Are you sure you want to change the base?

Added Modbus to OpenDTU #582

Conversation

ArekKubacki commented Feb 13, 2023

ArekKubacki commented Feb 27, 2023 • edited

PhiDiWoe commented Mar 4, 2023

horfic commented Mar 8, 2023 • edited

mzurhorst commented May 1, 2023

ArekKubacki commented Jun 5, 2023

AloisKlingler commented Jun 7, 2023

ArekKubacki commented Jun 7, 2023 • edited

AloisKlingler commented Jun 7, 2023 • edited

AloisKlingler commented Jun 9, 2023

AloisKlingler commented Jun 9, 2023

AloisKlingler commented Jun 21, 2023

MIC1981 commented Jun 30, 2023

fantilator commented Jul 7, 2023

madmartin commented Jul 18, 2023 • edited

mschimpfhuber commented Aug 18, 2023

heppth commented Oct 30, 2023

stefan123t commented Oct 30, 2023

ArekKubacki commented Nov 6, 2023

stefan123t commented Nov 6, 2023

ArekKubacki commented Nov 9, 2023

pneutek commented Nov 19, 2023

hwinkel commented Dec 17, 2023

AloisKlingler commented Dec 18, 2023

stefan123t commented Dec 18, 2023

AloisKlingler commented Dec 18, 2023

hwinkel commented Dec 18, 2023

ArekKubacki commented Feb 27, 2023 •

edited

horfic commented Mar 8, 2023 •

edited

ArekKubacki commented Jun 7, 2023 •

edited

AloisKlingler commented Jun 7, 2023 •

edited

madmartin commented Jul 18, 2023 •

edited