search

hassio-addons / addon-zwave-js-ui

Z-Wave JS UI - Home Assistant Community Add-ons
https://addons.community
MIT License
130 stars 36 forks source link

Temperature Control issue #714

Open feradelectronics opened 1 month ago

feradelectronics commented 1 month ago

Problem/Motivation

When I try to change temperatures on my thermostat i get the following out of range error.

Expected behavior

I am using Celsius for temperatures but looks like it is expecting Fahrenheit.

Actual behavior

Seems to expect set temperature in a Fahrenheit value which is too high for Celsius conversion

Steps to reproduce

z-wave device add temp card on lovelace page and try to increment or decrement change set value in Celsius. See https://github.com/home-assistant/core/issues/123612

Proposed changes

Distinguish and convert min max values between Celsius and Fahrenheit temperature values. Limits may need to be adjusted for different applications.

github-actions[bot] commented 2 weeks ago

There hasn't been any activity on this issue recently, so we clean up some of the older and inactive issues. Please make sure to update to the latest version and check if that solves the issue. Let us know if that works for you by leaving a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thanks!

nconder commented 1 week ago

Same issue but my temperature units are reversed. I am using Fahrenheit for temperatures but looks like it is expecting Celsius.