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

data from SDM120M incomplete #340

Open
sicheste opened this issue Mar 10, 2024 · 7 comments
Open

data from SDM120M incomplete #340

sicheste opened this issue Mar 10, 2024 · 7 comments

Comments

@sicheste
Copy link
Contributor

sicheste commented Mar 10, 2024

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
Copy link
Contributor Author

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
Copy link
Contributor Author

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:

  • The detection is not working (at least here).
  • The names are constructed using IDs, which are based on the alphabetical order of the meters. This makes the naming unstable when adding / removing meters.

@isarrider
Copy link
Contributor

@andig
Copy link
Contributor

andig commented Jun 20, 2024

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

@isarrider
Copy link
Contributor

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
Copy link
Contributor

andig commented Jun 20, 2024

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

@isarrider
Copy link
Contributor

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

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

No branches or pull requests

3 participants