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

DSMR hourly gas consumption incorrect #20097

Closed
wburgers opened this Issue Jan 14, 2019 · 0 comments

Comments

Projects
None yet
2 participants
@wburgers
Copy link
Contributor

wburgers commented Jan 14, 2019

Home Assistant release with the issue:

Latest (0.85.1)

Last working Home Assistant release (if known):
Unknown (this was probably always incorrect?)

Operating environment (Hass.io/Docker/Windows/etc.):

Docker (https://hub.docker.com/r/lroguet/rpi-home-assistant/)

Component/platform:

https://www.home-assistant.io/components/sensor.dsmr/

Description of problem:
I have a DSMR 5 Smart meter and get telegrams from it using ser2net.
In the DSMR sensor in home assistant, a derivative property is calculated from the gas meter readings.
This property is the hourly gas usage in m3/h.
https://github.com/home-assistant/home-assistant/blob/master/homeassistant/components/sensor/dsmr.py#L305
I used somewhere around 4 m3 of gas to heat my apartment yesterday, but the hourly gas consumption barely goes above 0.1 m3/h.
See the pictures below.

Calculation example:
Suppose I would use 0.1 m3/h at a constant rate.
After 24 hours that would add up to 2.4 m3.
As I use almost 4 m3 and the gas flow has a mean of about 0.04 m3/h, I conclude that the m3/h rate is incorrectly calculated.
Unless my high school math fails me?

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

sensor:
  - platform: dsmr
    port: 9999
    host: 192.168.0.33
    dsmr_version: 5

Traceback (if applicable):


Additional information:
https://photos.app.goo.gl/TXMZGL82XnAwgu8p6
https://photos.app.goo.gl/mRGikAK7EwA7NU1aA

I may be able to create a PR later, but if someone else gets to it first it is greatly appreciated.

@wburgers wburgers referenced this issue Mar 3, 2019

Merged

fix derived rate, fixes #20097 #21620

3 of 9 tasks complete

@wafflebot wafflebot bot added the in progress label Mar 3, 2019

@wafflebot wafflebot bot removed the in progress label Mar 3, 2019

amelchio added a commit that referenced this issue Mar 3, 2019

fix derived rate, fixes #20097 (#21620)
* fix derived rate, fixes #20097

* fix derived rate, fixes #20097

* Fix typo

thnx @amelchio

* Make the test more realistic

Took values from my own smart meter for the test

* Update test to ignore rounding issues

mxworm added a commit to mxworm/home-assistant that referenced this issue Mar 4, 2019

Merge branch 'dev' into current
* dev: (344 commits)
  Better output of workflow
  Upgrade youtube_dl to 2019.03.01 (home-assistant#21647)
  GitHub Workflow (home-assistant#21643)
  Fix actions with tox (home-assistant#21642)
  Replace travis (home-assistant#21641)
  Allow configuration of update interval for ness_alarm (home-assistant#21415)
  Updated frontend to 20190303.0
  Update translations
  Add storage helper to ZHA and use it for the device node descriptor (home-assistant#21500)
  Log exception occurred in WS service call command (home-assistant#21584)
  Fix calc next (home-assistant#21630)
  Expose create/delete cloudhook (home-assistant#21606)
  Return Netatmo climate operation_mode instead of boiler status (home-assistant#21633)
  Bump pyflunearyou to 1.0.3 (home-assistant#21634)
  fix derived rate, fixes home-assistant#20097 (home-assistant#21620)
  Add optional sender name for SendGrid (home-assistant#21610)
  Add 'app_name' property and 'apps' config entry to Fire TV (home-assistant#21601)
  Add device HMIP-eTRV-C to HomematicIP (home-assistant#21612)
  Bumping aioasuswrt (home-assistant#21627)
  SmartThings remove SmartApp/Automation on integration removal (home-assistant#21594)
  ...

@wburgers wburgers referenced this issue Mar 8, 2019

Merged

Fix TypeError #21734

3 of 9 tasks complete

@balloob balloob referenced this issue Mar 20, 2019

Merged

0.90.0 #22216

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.