volkszaehler / mbmd

ModBus Measurement Daemon - simple reading of data from ModBus meters and grid inverters
BSD 3-Clause "New" or "Revised" License
224 stars 81 forks source link

data from SDM120M incomplete #340

Open sicheste opened 5 months ago

sicheste commented 5 months ago

If I add the SDM120M as SDM220 to the config, I will not get all values. Especially the Power (W) is missing. If I add it as a SDM I get this value.

As SDM220: image

As SDM: image

Btw: What is the name field in the config for? The name (here SDM2201.3 and SDM1.3 is constructed by type + position in the alphabetically sorted list of sensors + . or - + bus id. Thus the names in the API and MQTT are unstable.

Btw2: docker run volkszaehler/mbmd scan -a 192.168.2.11:502 did not detect the SDM120.

sicheste commented 5 months ago

I just recognized that the current docker image is already 4 years old. I will try in the next days to build my own one using the current master branch and give it a try. Maybe it is time for a new version :)

sicheste commented 5 months ago

So, I had some time. I have created a new docker image locally, based on the current master branch. Running mbmd scan was not detecting the SDM120M. But manually giving it as SDM220:3 (it is number 3 on the bus) it works. Data in MQTT and on the web interface are fine.

So, still open:

isarrider commented 1 month ago

https://hub.docker.com/r/isarrider/mbmd/tags try mine...

andig commented 1 month ago

@isarrider am I missing an open PR that did not get merged?

isarrider commented 1 month ago

nah, I just wanted to give @sicheste and others a docker image that isnt old... (actually thats why I started also building mine, bc there is no newer)

andig commented 1 month ago

Thank you. Would be happy for build workflow PR- don't have the time...

isarrider commented 1 month ago

I know the feeling but I am always hoping for the young IT Students or similar...