gysmo38 / mitsubishi2MQTT

Mitsubishi to MQTT with ESP8266 module
GNU Lesser General Public License v2.1
393 stars 141 forks source link

Minimum and maximum temperature setting is only in C #47

Closed shampeon closed 4 years ago

shampeon commented 4 years ago
Screen Shot 2020-04-10 at 1 17 52 PM

In the Advanced settings, setting the unit to Fahrenheit doesn't affect the minimum/maximum settings. For example, if I set the unit to Fahrenheit and then set the minimum temperature to 61 degrees, the actual setting is 142 degrees. If I set the maximum temperature to 88 degrees, the actual setting is 190 degrees.

From the HA autodiscovery topic:

"min_temp":142,
   "max_temp":190,

Changing the unit should change the minimum/maximum temperature display, and the conversion should go both ways.

Internally and in the config files, min_temp and max_temp can remain Celsius values. For display, however, they need to match the unit.

gysmo38 commented 4 years ago

Resolved by @shampeon :)