search

homewizard / python-homewizard-energy

HomeWizard Energy API implementation in Python. Designed for Home Assistant but can be used for any Async application.
https://api-documentation.homewizard.com
Apache License 2.0
14 stars 4 forks source link

L/pm calculations #407

Closed baudneo closed 2 months ago

baudneo commented 2 months ago

I don't use this repo but, a user on the Hass forum does and is having an issue with the L/pm sensor from the water meter reader.

They are saying that when there are 0 pulses per minute, the lpm sensor is sending erroneous data of 0 or 1.27 L/pm, this plays havoc on Hass automations that are time based; if lpm <= 0 for x minutes, automations never fire due to the sensor bouncing between 0 and 1.27 even when no water is flowing. See this Hass forum post.

Can you point me towards where in the code the lpm calculations are performed?

Also, it would be beneficial to expose the pulse counter for users who want tighter control over their stats.

TIA and thanks for your work!

DCSBL commented 2 months ago

Hi!

The calculation is handled in the device firmware so the code to that is not available. This is the first time we've encountered this issue. It might be worth directing the user to HomeWizard Support to check if the device is faulty.

This library serves as a bridge between the device and its implementation (typically Home Assistant). Any filtering should be handled within the implementation.

We can investigate if we have to add some filtering in Home Assistant, but we'll need to collect more cases to assess how many users are affected.

baudneo commented 2 months ago

Awesome, thanks. I'll forward this info and advise the user to reach out to support for further troubleshooting.

baudneo commented 2 months ago

I'll close the issue as I'm out of the loop. Hopefully the user reaches out! Thanks!