core: Unable to toggle Daikin switches

The problem

I am unable to toggle switches from Daikin integration. These switches correspond to zones in my A/C system.

What version of Home Assistant Core has the issue?

2023.6.2

What was the last working version of Home Assistant Core?

2023.6.1

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Daikin

Link to integration documentation on our website

https://www.home-assistant.io/integrations/daikin

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

2023-06-15 20:53:20.253 ERROR (MainThread) [homeassistant.components.websocket_api.http.connection] [547110915920] DaikinAirBase.set_zone() takes 3 positional arguments but 4 were given
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/websocket_api/commands.py", line 205, in handle_call_service
    await hass.services.async_call(
  File "/usr/src/homeassistant/homeassistant/core.py", line 1910, in async_call
    task.result()
  File "/usr/src/homeassistant/homeassistant/core.py", line 1950, in _execute_service
    await cast(Callable[[ServiceCall], Awaitable[None]], handler.job.target)(
  File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 226, in handle_service
    await service.entity_service_call(
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 811, in entity_service_call
    future.result()  # pop exception if have
    ^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1034, in async_request_call
    await coro
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 851, in _handle_entity_call
    await result
  File "/usr/src/homeassistant/homeassistant/components/daikin/switch.py", line 97, in async_turn_off
    await self._api.device.set_zone(self._zone_id, "zone_onoff", "0")
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: DaikinAirBase.set_zone() takes 3 positional arguments but 4 were given


### Additional information

_No response_

About this issue

  • Original URL
  • State: closed
  • Created a year ago
  • Reactions: 4
  • Comments: 50 (6 by maintainers)

Commits related to this issue

Most upvoted comments

Very grateful to you for all your hard work. I know what it’s like supporting an opensouce project. Been there, done that. In this instance, my life will go on with zones I have to use the app to change OR get up off my arse and turn on and off manually until a fix is produced. My very best respect to you lot.

Why is it being left broken? I don’t understand.

We, the developers of the Daikin integration are doing everything for free, there are other priorities in life than supporting open source projects and my Daikin unit (that currently is not even installed) does not support zones. We as community have to work together just like @mover85 have been doing lately 🎉

I’m sorry for the delay. I was trying to get a new version of pydaikin released but found an issue with unique id not being correct.

Daikin Becomes unavailable after the new core update and new entity appears. Had to remap to that entity however switches for zones dont work.

Hi all, can confirm that the just released update of 2023.7.2 has fixed this issue. Thank you @mover85 and @fredrike.

I’m confused at how it works, because it says here that it’s been merged. I’m not sure what version it’ll go into though.

#95656

Thanks for all the effort to those trying to get this fixed!

I think #95656 is a merge to the dev branch only, not yet any of the releases. I’ve been keeping a close eye on the full changelog for 2023.7 and don’t see Daikin mentioned yet so have been holding off before upgrading.

I’m confused at how it works, because it says here that it’s been merged. I’m not sure what version it’ll go into though.

https://github.com/home-assistant/core/pull/95656

Thanks for all the effort to those trying to get this fixed!

Reverted to 2023.6.1, can’t afford to live with broken integration.

On Fri, 23 June 2023, 7:45 am hondoaus, @.***> wrote:

Sam issue and waiting for a fix.

How long does it usually take for the fixes to be released?

TypeError: DaikinAirBase.set_zone() takes 3 positional arguments but 4 were given

Home Assistant 2023.6.2 Supervisor 2023.06.2 Operating System 10.3 Frontend 20230608.0 - latest

— Reply to this email directly, view it on GitHub https://github.com/home-assistant/core/issues/94648#issuecomment-1603355277, or unsubscribe https://github.com/notifications/unsubscribe-auth/AVKPZHTO5AD52ENTUR3CDDLXMS4IDANCNFSM6AAAAAAZHVF3EI . You are receiving this because you commented.Message ID: @.***>

Can confirm this is an issue for me too since 2023.6.2