Wrong zone set in SKYFi

[Fredrik Erlandsson]

See here: https://community.home-assistant.io/t/daikin-skyfi-and-0-111-0-first-time-use-no-zones-detected/203900/46

‌

@Dat Peter

[Dat Peter]

I don’t have zone, my zone are manual control via physical switch. Let me poke around and see what I can find.

[Dat Peter]

Didn’t realise there is thread about SkyFi, will read and try to understand it first.

[Dat Peter]

Something is wrong with the latest master branch output, i just switched 48fb530a (Bump version: 2.1.2 -> 2.1.3), and the output is expected with the zones.

Master branch zone list output is:

[('Zone 1', '0')]

Whereas the other working commit is:

[('Zone 1', '0'), ('Zone 2', '0'), ('Zone 3', '0'), ('Zone 4', '0'), ('Zone 5', '0'), ('Zone 6', '0'), ('Zone 7', '0'), ('Zone 8', '0')]

‌

Still investigating.

[Fredrik Erlandsson]

I think that it should be a +1 in the set zone method.

[Dat Peter]

Nevermind with my above comment, i failed at using git

Yep, looking at how the zone_id get pass around.

Whirlpool page mentioned, Zone1 should be z=2, Zone2 should be z=3, etc…

[Fredrik Erlandsson]

So +2 then?

‌

The forum looks like we are low by just one.

[Fredrik Erlandsson]

Fixed in https://bitbucket.org/mustang51/pydaikin/pull-requests/36/fix-set-zone