home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
73.45k stars 30.69k forks source link

Add zone support for Daikin Airbase #22376

Closed jurgenweber closed 5 years ago

jurgenweber commented 5 years ago

Home Assistant release with the issue:

Last working Home Assistant release (if known): Hass 0.90 does not have support. Conversation here: https://community.home-assistant.io/t/support-for-daikin-airbase-units/102622/26

I spoke to Fredrike on discord and I tested his code that is in dev against my unit. All works well. We are looking at adding zone support as per this post https://whrl.pl/RfiF8K.

Operating environment (Hass.io/Docker/Windows/etc.): hassbian, etc

Component/platform: https://www.home-assistant.io/components/daikin/

Description of problem:

Add zone support, as per the whirlpool thread I have confirmed that the commands work.

$ curl "http://192.168.1.125/skyfi/aircon/get_zone_setting"
ret=OK,zone_name=%4d%61%73%74%65%72%3b%4b%69%74%63%68%65%6e%3b%48%61%75%6e%74%65%64%3b%4f%66%66%69%63%65%3b%47%75%65%73%74%3b%4c%6f%75%6e%67%65%3b%4e%75%72%73%65%72%79%3b%20%20%20%20%20%20%20%5a%6f%6e%65%38%2c,zone_onoff=0%3b0%3b0%3b0%3b0%3b1%3b1%3b0

Now setting the zone is a little tricky,

curl 'http://192.168.1.125/skyfi/aircon/set_zone_setting?|pw=&zone_name=%4d%61%73%74%65%72%3b%4b%69%74%63%68%65%6e%3b%48%61%75%6e%74%65%64%3b%4f%66%66%69%63%65%3b%47%75%65%73%74%3b%4c%6f%75%6e%67%65%3b%4e%75%72%73%65%72%79%3b%20%20%20%20%20%20%20%5a%6f%6e%65%38%2c&zone_onoff=%31%3b%30%3b%30%3b%30%3b%30%3b%31%3b%31%3b%30'

Here is a break down of hte payload "zone_name=&zone_onoff=<url encoded list of zones 0 or 1>"

I did try without encoding, encoding everything but this combination seems to be the only one that works.

jurgenweber commented 5 years ago

@fredrike

fredrike commented 5 years ago

The idea is to implement each zone as a separate switch.

fredrike commented 5 years ago

I'll start working on this once #22382 have been merged.

awarecan commented 5 years ago

We are not asking to create an issue in order to create a pull request.

No need to create this next time.