search

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
73.3k stars 30.62k forks source link

Screenlogic Integration: GPM value erroneous for type VS Pump #51678

Closed MrBill12 closed 3 years ago

MrBill12 commented 3 years ago

The problem

Type VS pumps have RPM and Watts attributes. There is no value for GPM value for type VS pumps.

Home Assistant reports 255 GPM regardless of speed setting for type VS pump. The correct value should be either null (or possibly 0).

In the attached screenshot collage, Home Assistant values are on the left, the right is a corresponding screenlogic screencap. The problem is that HA displays 255GPM instead of null (or 0). (note: the fact that watts deviate slightly is due to the timing on the screenshots and can be ignored.)

(Type VSF pumps have the GPM value and it works correctly)

Pentair Issue Screencaps

What is version of Home Assistant Core has the issue?

2021.6.2

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

Pentair ScreenLogic

Link to integration documentation on our website

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

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

Device Info

probot-home-assistant[bot] commented 3 years ago

Hey there @dieselrabbit, mind taking a look at this issue as its been labeled with an integration (screenlogic) you are listed as a codeowner for? Thanks! (message by CodeOwnersMention)

dieselrabbit commented 3 years ago

Thanks for the report. Looking into it.