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
71.02k stars 29.65k forks source link

Ambient Weather Station Integration Doesn't Include Hourly Rain #121886

Open dan-orth opened 1 month ago

dan-orth commented 1 month ago

The problem

My Ambient Weather Station (WS-5000) reports hourly rain on my physical weather station console, however there is no Hourly Rain sensor that I can find within the integration.

I am trying to get up an automation based on hourly rain totals.

What version of Home Assistant Core has the issue?

core-2024.7.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

ambient_station

Link to integration documentation on our website

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

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

@bachya – Forgive me if this isn't a bug and instead by design or due to a limitation from Ambient Weather. I am just getting started with Home Assistant and automations and was hoping to use hourly rain in several automations to park my robotic mower when the grass was too wet and start the automower again after it had stopped raining.

While I believe I can use Last Rain to help restart the mower, Hourly Rain seems like the best way to determine when to pause (e.g., Daily Rain may be misleading if it's a drizzle all-day or resets at midnight mid-storm, Rain Intensity may spike due to a sudden quick-passing storm, and Event Rain lasts too long (24hr) before resetting).

Thanks for the great integration!

home-assistant[bot] commented 1 month ago

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

Code owner commands Code owners of `ambient_station` 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 ambient_station` 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)


ambient_station documentation ambient_station source (message by IssueLinks)

bachya commented 1 week ago

Thank you for your patience. Can you send me diagnostics info? That will tell us whether that data is coming through via the API.

dan-orth commented 1 week ago

@bachya Does this diagnostic data help? I see there: "hourlyrainin": 0, though I do not see a corresponding entity for Hourly Rain.

Thanks for looking into this!

config_entry-ambient_station-01J2AV7D55FRBS35ZCZAPX5JPB.json

bachya commented 1 day ago

Thanks for your patience. Your diagnostics do show that the data is coming through. hourlyrainin is a supported sensor though: https://github.com/home-assistant/core/blob/d68ee8dceae7acd2fbdc2be0724847ac0827dcde/homeassistant/components/ambient_network/sensor.py#L130-L137

Can you show me a screenshot of what sensors are coming through your integration? That may give us another clue.