home-assistant/core

Huum Integration does not support fahrenheit

kneath opened this issue · 2 comments

kneath commented

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 commented

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

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