sman591 / homebridge-lg-thinq-ac

Homebridge plugin for LG ThinQ-enabled portable air conditioner
Apache License 2.0
53 stars 16 forks source link

Temperature should optionally be available as a separate sensor in HomeKit #41

Closed srshearer closed 4 years ago

srshearer commented 4 years ago

The AC unit's current temperature is not exposed as a temperature sensor in HomeKit. It would be ideal if there was an optional config parameter to add a the current temperature as a separate sensor so that it could be used for automations. A similar approach was taken to expose various sensors in this Dyson plugin.

sman591 commented 4 years ago

I implemented this a few months back and determined the Home app somehow didn't allow sensor-driven automation for the sensor I exposed. The Dyson implementation looks very similar though so I'm not sure what happened; can give it another shot

sman591 commented 4 years ago

@srshearer Could you attach a screenshot of what sensors you see with the Dyson plugin? I tried this again and am not seeing anything on the "sensor automation" screen despite activating a temperature sensor.

srshearer commented 4 years ago

Strange. First of all you are right, I do not see any temperature sensors. What I do see is a sensor for air quality. I must have seen the Dyson fan as an option for automation sensors and made an assumption.

IMG_3351 IMG_3353

sman591 commented 4 years ago

Aha, yeah that's very strange.

I'll keep a branch open in case this changes for future iOS versions, but until then, I'm going to close this.

Note other temperature-based actions can be created using the Shortcuts app if/else conditions, though you can't have temperature be a trigger for that unless Apple implements this. Time/occupancy/Siri/etc can still be a trigger though