lurgh / infinitive

Infinitive impersonates a SAM on a Carrier Infinity system management bus. This fork supports multi-zone systems and MQTT.
MIT License
7 stars 1 forks source link

Interface does not differentiate between Celsius and Farenheit #3

Closed maximvoven closed 2 months ago

maximvoven commented 8 months ago

MQTT and JSON API do not state the units used for temperature. This makes home assistant bug out depending on how the data is interpreted depending on which config provides the data. Bus logs coming soon. Screenshot 2024-02-18 at 6 59 11 PM Screenshot 2024-02-18 at 7 05 47 PM

maximvoven commented 8 months ago

First Batch of Logs: 17:15 switched from C to F 17:17 switched from F to C resplog.24021819.txt

lurgh commented 8 months ago

Thanks for getting in touch about this. I am not clear yet what exactly is going wrong for you, other than maybe that the UI card is showing in °F.

My Inifinty system is set to °F and that's the default for my HomeAssistant. I don't think infinitive has temp units baked in but there could be something going on. Best I recall, the serial API is consistently just using the units that the system is set to, and we are passing the values through as we see them. If needed, we could detect the temp units from the system and label the values to/from MQTT but I don't know if it is necessary or how the mqtt climate integration would react.

Maybe try setting temperature_unit: C in the mqtt climate definitions and see if that stabilizes it for you? Or let me know more about what's happening on the HA side and we can figure it out.

lurgh commented 2 months ago

closing due to lack of response