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.74k stars 30.85k forks source link

Huum Integration does not support fahrenheit #129012

Open kneath opened 3 weeks ago

kneath commented 3 weeks ago

The problem

When using fahrenheit for Home Assistant, the Huum integration assumes the temperature is always celsius and limits to 40-110, and does not convert to test the limits. For example, it is impossible to set a temperature of 120˚F since it assumes it is 120˚C for the limits. The Climate widget itself is correct, and allows me to change between 104˚F-230˚F. The API call itself is correct, and I can use the Sauna between 104˚F-110˚F.

The logs show it converting, but trial and error shows it will only allow fahrenheit values.

What version of Home Assistant Core has the issue?

core-2024.9.0

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant Container

Integration causing the issue

huum

Link to integration documentation on our website

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

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

Logger: homeassistant.components.huum.climate
Source: components/huum/climate.py:133
integration: Huum (documentation, issues)
First occurred: 5:28:16 PM (15 occurrences)
Last logged: 8:30:55 PM

    Temperature '81.66666666666667' must be between 40-110
    Temperature '81.11111111111111' must be between 40-110
    Temperature '80.55555555555556' must be between 40-110
    Temperature '44.44444444444444' must be between 40-110
    Temperature '47.22222222222222' must be between 40-110


### Additional information

_No response_
home-assistant[bot] commented 3 weeks ago

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

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


huum documentation huum source (message by IssueLinks)

frwickst commented 3 weeks ago

Thanks for the info @kneath! I'll try to find the time to look into this as soon as possible.