aivus / com.gree

Control your Gree, Cooper&Hunter HVAC devices using Athom Homey
https://apps.athom.com/app/com.gree
GNU General Public License v3.0
8 stars 5 forks source link

HomeyKit #44

Closed jahnkenneth closed 4 years ago

jahnkenneth commented 4 years ago

Hello, This app also works for Wilfa AC. However it does not show up in my HomeKit setup. Is this possible to fix? I am using the HomeyKit app.

aivus commented 4 years ago

Hey @jahnkenneth

Thanks. I've added it to the supported list.

Unfortunately I don't have Apple devices and cannot test HomeKit. What exactly doesn't work?

jahnkenneth commented 4 years ago

The HomeyKit-app doesn’t recognize it as a thermostat. Maybe @robertklep can advise? :-)

robertklep commented 4 years ago

The problem is that Homeykit only accepts thermostat devices that have both measure_temperature and target_temperature (these are both required capabilities for Homekit), and the Gree driver only has the latter :(

jahnkenneth commented 4 years ago

@aivus Is this something that you could implement?

aivus commented 4 years ago

@jahnkenneth ACs are not providing current temperature.

I'll check what I can do here.

jahnkenneth commented 4 years ago

My AC provides the current temperature in the EWPE app... I have also gotten it to work in HomeBridge, where it also provides the current temperature. Also my Daikin AC works with HomeyKit...

aivus commented 4 years ago

My AC provides the current temperature in the EWPE app

That's interesting. Are you sure that it shows current information, not target?

I'm using a 3rd party library for interaction with AC and it doesn't have any options to get current temp from such ACs.

aivus commented 4 years ago

I'll check adding measure_temperature to the driver and use the same temperature as in target_temperature.

@jahnkenneth I'll ask you to test this version before release to confirm that it works. Ok? Please share with me email address that you are using for homey (please use my email from the github profile). It allows me to push alpha version to you.

jahnkenneth commented 4 years ago

My AC provides the current temperature in the EWPE app

That's interesting. Are you sure that it shows current information, not target?

I'm using a 3rd party library for interaction with AC and it doesn't have any options to get current temp from such ACs.

Yes, I am very sure. :-)

aivus commented 4 years ago

Based on the sources of Homebridge app parameter TemSen can be used as a measured temperature.

But my HVAC reports 65 °C always (I believe it doesn't temperature sensor). Testing in progress

All possible information from EWPE: https://github.com/tomikaa87/gree-remote/issues/7

grunci commented 4 years ago

Hi @aivus , Just substract 40 from the reported TemSen, and you will see the actual temperature in celsius. I am investigating further in the area.

aivus commented 4 years ago

Hey @grunci.

It's already investigated and pushed to the Homey's test channel: https://homey.app/en-us/app/com.gree/Gree/test/

Changes: https://github.com/aivus/com.gree/pull/49 and https://github.com/inwaar/gree-hvac-client/pull/11

Will be released 🔜

aivus commented 4 years ago

Done in v0.4.0