home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
72.51k stars 30.34k forks source link

Wrong power-values with "DZG DVS74"-Meter #90651

Closed Chr1bu closed 1 year ago

Chr1bu commented 1 year ago

The problem

Hello together,

i've a problem with the EDL21-Integration and my "DZG DVS74"-Meter. It shows wrong/negative values when the power is higher than ~400W(Workaround mentioned below) or ~2000W (temporarily stopps recording!).

What version of Home Assistant Core has the issue?

2023.3.6

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

edl21

Link to integration documentation on our website

https://www.home-assistant.io/integrations/edl21

Diagnostics information

N/A

Example YAML snippet

sensor:
  - platform: edl21
    name: SmartMeter
    serial_port: /dev/ttyUSB0

Anything in the logs that might be useful for us?

Nothing

Additional information

Meter Model: "DZG DVS7412.1" Meter Serialno: starts with "DZG00"

My search shows up that there is a known "encoding bug" in this type of meter:

VZLogger-Site (German only; yellow warning fields): https://wiki.volkszaehler.org/hardware/channels/meters/power/edl-ehz/dzg_dvs74

HomeAssistant-Community (Shows all the workarounds and tried fixes) https://community.home-assistant.io/t/edl21-showing-wrong-values-partially/330502/

This post "workarounds" my power-values higher ~400W... https://community.home-assistant.io/t/edl21-showing-wrong-values-partially/330502/6

...but, power-values higher than ~2000W wasn't fixed and the "sensor.sum_active_instantaneous_power doesn’t get updated at all" https://community.home-assistant.io/t/edl21-showing-wrong-values-partially/330502/10

There is also a running feature request for this, but i think that this should be a problem-/bugfix, rather than a feature request: https://community.home-assistant.io/t/work-around-for-dzg-smart-meter-firmware-error/342349 Therefore i have created this issue.

@mtdcr It would be nice when you can implement this workaround into your pysml-library. https://github.com/jmberg/libsml/commit/81c4026e3d94f7a384cdd89f62a727b83269cdec

@StephanU As soon as the power-value is under ~2000W again the sensor updates again, with no problems. Strange... Maybe the workaround mentioned above can also fix this part of the problem (with no info in the logs)?!

Thank you in advance. :) Chr1Bu

home-assistant[bot] commented 1 year ago

edl21 documentation edl21 source

moritzbeck13 commented 1 year ago

Neither do I have merge rights nor am I a maintainer here, so not sure why you keep tagging me.

Chr1bu commented 1 year ago

Neither do I have merge rights nor am I a maintainer here, so not sure why you keep tagging me.

Sorry, edited it.

mtdcr commented 1 year ago

Please open an issue in pysml including raw data. If you don't want to disclose your meter's ID to the public, you can send it by mail (obi at saftware dot de).

You should be able to capture data to a file using simple tools like cat or dd, once the serial port has been configured. Make sure Home Assistant does not use it at the same time.

Am 1. April 2023 22:59:47 MESZ schrieb Chr1bu @.***>:

Neither do I have merge rights nor am I a maintainer here, so not sure why you keep tagging me.

Sorry, edited it.

-- Reply to this email directly or view it on GitHub: https://github.com/home-assistant/core/issues/90651#issuecomment-1493119237 You are receiving this because you were mentioned.

Message ID: @.***>

Chr1bu commented 1 year ago

Good morning,

@StephanU The problem was hopefully fixed in the new pysml-release. Can you please bump it to 0.0.10 in the EDL21-Integration? I'll let you know if this fix/release is working, after it finally lands in the next HA-Release. :)

@mtdcr thank you very much so far.

Bye..

Chr1bu commented 1 year ago

Hello @mtdcr and @StephanU , i think, the new pysml-release fixed my(!!) issue, but raises a new problem described here: https://github.com/home-assistant/core/issues/91494

Maybe we can hold this issue open until both issues were fixed?!

StephanU commented 1 year ago

@Chr1bu It seems 2023.5.0 solved this problem :) Could you please close this issue.

Chr1bu commented 1 year ago

@StephanU Yes, it's working everything for me, now! Thanks. 👍 @mtdcr Thank you for your work and fixes!