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
74.18k stars 31.15k forks source link

Helper "Generic hygrostat" can not use sensor from another helper combined-sesor (min_max) #132161

Open trondtorsoe opened 21 hours ago

trondtorsoe commented 21 hours ago

The problem

When trying to move generic thermostats and hygrostats from configuration.yaml ---> helpers - the "Generic hygrostat" will not accept "sensor.moist_bathroom". This sensor is also a result of a helper "combined several sesors". All other sensors are availible....

The problem is not big and important. I can keep the setup in configuration.yaml - but still a tiny "error" :-)

What version of Home Assistant Core has the issue?

core-2024.12.0b4

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

Helpers - combine several sensors

Link to integration documentation on our website

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

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

Not a developer - but seems to me like the combined sensor helper is missing "Device class" ? 2024-12-03 11 51 27

home-assistant[bot] commented 21 hours ago

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

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


generic_hygrostat documentation generic_hygrostat source (message by IssueLinks)

trondtorsoe commented 20 hours ago

I think it should be an option to set Device class here : https://www.home-assistant.io/integrations/min_max

OR

possible to "allow non-humidity sensors" in the hygrostat-helper 😃