Ghawken / HomeKitLink-Siri

8 stars 3 forks source link

Thermostat not fully synced #15

Open compwiz777 opened 2 years ago

compwiz777 commented 2 years ago

Able to add Thermostat to Apple Homekit. Able to modify Mode and Temp. Homekit shows it as Fahrenheit which is selected in Bridge

Issues:

Device Details: batteryLevel : None buttonGroupCount : 5 configured : True coolIsOn : False coolSetpoint : 75.0 dehumidifierIsOn : False description : deviceTypeId : displayStateId : temperatureInputsAll displayStateImageSel : HvacCoolMode displayStateValRaw : 75 displayStateValUi : 75 enabled : True energyAccumBaseTime : None energyAccumTimeDelta : None energyAccumTotal : None energyCurLevel : None errorState : fanIsOn : False fanMode : Auto folderId : 0 globalProps : MetaProps : (dict) com.GlennNZ.indigoplugin.HomeKitLink-Siri : (dict) HomeKit_audioSelector : false (bool) HomeKit_bridgeUniqueID : 489474 (integer) HomeKit_deviceSensor : (string) HomeKit_deviceSubtype : Thermostat (string) HomeKit_inverseSelector : false (bool) HomeKit_publishDevice : true (bool) HomeKit_tempSelector : true (bool) homekit-name : Thermostat (string) heatIsOn : False heatSetpoint : 65.0 humidifierIsOn : False humidities : [45.0] humiditySensorCount : 1 hvacMode : Cool id : 194602244 lastChanged : 2022-05-28 19:01:03 lastSuccessfulComm : 2022-05-28 19:01:03 model : Thermostat (2441TH) name : Main Floor Thermostat ownerProps : emptyDict : (dict) pluginId : pluginProps : emptyDict : (dict) protocol : Insteon remoteDisplay : True sharedProps : com.indigodomo.indigoserver : (dict) states : States : (dict) humidityInput1 : 45 (real) humidityInputsAll : 45 (string) hvacCoolerIsOn : off (on/off bool) hvacDehumidifierIsOn : off (on/off bool) hvacFanIsOn : off (on/off bool) hvacFanMode : 0 (integer) hvacFanMode.ui : auto on (string) hvacFanModeIsAlwaysOn : false (bool) hvacFanModeIsAuto : true (bool) hvacHeaterIsOn : off (on/off bool) hvacHumidifierIsOn : off (on/off bool) hvacOperationMode : 2 (integer) hvacOperationMode.ui : cool on (string) hvacOperationModeIsAuto : false (bool) hvacOperationModeIsCool : true (bool) hvacOperationModeIsHeat : false (bool) hvacOperationModeIsOff : false (bool) hvacOperationModeIsProgramAuto : false (bool) hvacOperationModeIsProgramCool : false (bool) hvacOperationModeIsProgramHeat : false (bool) setpointCool : 75 (real) setpointHeat : 65 (real) temperatureInput1 : 75 (real) temperatureInputsAll : 75 (string) subModel : subType : supportsAllLightsOnOff : False supportsAllOff : False supportsCoolSetpoint : True supportsHeatSetpoint : True supportsHvacFanMode : True supportsHvacOperationMode : True supportsStatusRequest : True temperatureSensorCount : 1 temperatures : [75.0] version : 15

Ghawken commented 2 years ago

Ok.

[edit]

  1. Can I double check what version running? As checking code base from afar and can’t find anything obvious as yet
  2. Also double check settings in Home iOS running application - go to iPhone/iPad, Settings,General, Language and Region check is F. (Would guess would have noticed this if wasn’t - but pays to check)

Thanks for the device details… That will be a bug - not helped as don’t have any Insteon devices to test, also thermostat support is still quite beta Will have a look in a few days when back at what can do - probably doesn’t recognise device (now that you have given device details will add and that may be root cause of a couple of these)

I think everything you are describing is probably the same lack of temperature conversion (there is a min/max temp range which is probably default to) and because of this lack of conversion is way above.

  1. I may need some more device logging to see what your Insteon thermostat sends - to do so need to set debug device in plugin config menu, which will show log details every time this device is changed. But probably not much point until get to the bottom of why temp conversion failing.
compwiz777 commented 2 years ago

1- I am running the latest update to indigo = 2022.1.1 / HomeKit Plugin ver = 0.3.1

2- Confirmed that iPhone settings are set to F

3- I understand this is still beta so no worries it doesn't all work right away. I am here to help whatever you need so we can get this fixed and consider this my part to help support this plugin I am taking advantage of 😉

Ghawken commented 2 years ago

Ok - thanks. Think have found the temp F/C conversion issue (away at moment)

compwiz777 commented 2 years ago

Thanks for the update.

Also in the future I would be happy to help with testing for this plugin. I have many Insteon devices and One Z-wave device that I can use to test with 😉

Ghawken commented 2 years ago

Found the issues - was converting F to C as needed on startup of HomeKit Apps.Fixed. & other issue is the TargetTemp doesn’t update when Cool Heat mode changes (auto - different) First done, and second - should be fixed in 24hrs or so. Will post new release

Ghawken commented 2 years ago

Try Release 0.3.2

compwiz777 commented 2 years ago

Installed the latest update and that seams to have fixed all of the issues.

Thanks for the great fix all appears to be well now. Let me know If you would like any device data / pics for reference and or confirmation of fix

7RDR7 commented 2 years ago

I'm using HKL for multiple thermostats (physical and virtual), and all are operating properly. The only issue that I have observed is that native HomeKit thermostat icons indicate the following states: "Cooling to 75" when the system is operating, and "Cool to 75" when the system has reached target temperature, the icon also displays a status color (blue-cool operating, orange-heat operating, green-target temperature), whereas my HKL instance always shows the thermostat "Cooling to" or "Heating to" along with the appropriate color (blue/orange) even if temperature has been reached and the system has stopped operating.