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

Hunter Hydrawise Flow Meter Info wrong #119819

Closed
cslphoenix opened this issue Jun 17, 2024 · 6 comments · Fixed by #119988
Closed

Hunter Hydrawise Flow Meter Info wrong #119819

cslphoenix opened this issue Jun 17, 2024 · 6 comments · Fixed by #119988

Comments

@cslphoenix
Copy link

The problem

Hello everyone,

The integration was not available for a few days, then new sensors were available.

The water flow meter is now recognized again, but the values ​​do not match what the Hunter Hydrawise app says.

What version of Home Assistant Core has the issue?

core-2024.6.3

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

Hunter Hydrawise

Link to integration documentation on our website

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

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

The water quantities cannot be correct.

image

https://app.hydrawise.com/config/events

image From Home Assistant history
@home-assistant
Copy link

Hey there @dknowles2, @ptcryan, mind taking a look at this issue as it has been labeled with an integration (hydrawise) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of hydrawise can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign hydrawise Removes the current integration label and assignees on the issue, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


hydrawise documentation
hydrawise source
(message by IssueLinks)

@dknowles2
Copy link
Contributor

@thomaskistler can you take a look at this?

@dknowles2
Copy link
Contributor

Oh, I think it's because we're not setting the unit of measurement correctly. It's hard-coded as gallons in the integration, but your Hydrawise controller is set to liters.

@cslphoenix
Copy link
Author

Hey, thanks for the quick reply.

Is there a trick to fix it or does my helper have to convert the result?

Thanks

@thomaskistler
Copy link
Contributor

We do have the unit in the water flow summary. We just need to set it properly in the integration. I will take a look.

@cslphoenix
Copy link
Author

Helllo,

on

FLOW_CONTROLLER_SENSORS: tuple[HydrawiseSensorEntityDescription, ...] = (

I'm not entirely sure about that.

Nice work guys

@github-actions github-actions bot locked and limited conversation to collaborators Jul 21, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants