fjhorrillo / homebridge-airzone-cloud

Homebridge Airzone Cloud
Apache License 2.0
22 stars 3 forks source link

°F gets extra conversion in Home #25

Closed sgeba closed 1 year ago

sgeba commented 1 year ago

I assume with the release of these in the US there is an issue with the use of °F instead of C. The temperature comes in to Home with all of the temperatures from the app converted again. Example - my AirzoneCloud is set for F and heat set to 68F. In the Home it treats the as a C temperature and converts it again to F resulting in showing 155F in Home.

I temporarily set my airzonecloud in C and restarted the child bridge. Everything comes in correctly in home as the proper F temperature but won't be able to leave it that way. Any help / data / debug you need let me know!

fjhorrillo commented 1 year ago

Hello @sgeba, could you send me the logs with the debug mode on in the plugin to check what is getting from your Airzone cloud, thanks in advance.

sgeba commented 1 year ago

homebridge-airzone.log.txt

don't know what data in here is sensitive if at all, I removed a few of the auth things, but the calls are all still there, let me know if you need something different.

sgeba commented 1 year ago

Was the attached data any help?

fjhorrillo commented 1 year ago

Hello, I checked your logs and I could see that your Airzone provide data in both unit (C & F). I saw an error setting the temperature with F and the error is regarding a maximum in C. I'm going to test it with my phone in F as I guest that it will be the problem. Let me some time, I will be back with news a son as possible.

arcticwolf18 commented 1 year ago

Hey there. Was just wondering if you had any input on this. Still an ongoing issue for me.

fjhorrillo commented 1 year ago

Hi! I assume you have configured the temperature unit by system preference and also in my home app, right? I was checking in this way and as I could see the problem is that the iPhone always uses celsius. I'm going to adapt the plugin in this way but I need a bit of time to change all to only work with celsius and remove the option to change the unit.

fjhorrillo commented 1 year ago

Fixed with version 0.3.3