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.32k stars 30.63k forks source link

Tado Issues with new features - Fan #121120

Open 03397 opened 4 months ago

03397 commented 4 months ago

The problem

Guys after the upgrade to the 2024.7.0 the tado features related with the fan are not 100% correct. For example my 2 Daikin A/C support 6 different fan modes + auto, I can see 6 different fan modes + auto inside the Tado application but inside Home Assistant I can see only low, medium, high, auto. Screenshot_20240705-083157

In addition when I turn off my A/Cs I am gettting an error message. This is happening because during turning off the A/C the fan turns to off image

Logger: homeassistant.components.websocket_api.http.connection Source: components/websocket_api/commands.py:283 integration: Home Assistant WebSocket API (documentation, issues) First occurred: 11:47:40 PM (1 occurrences) Last logged: 11:47:40 PM

[139733377552096] Fan mode is not valid. Valid fan modes are: medium, auto, low, high

What version of Home Assistant Core has the issue?

2024.7.0

What was the last working version of Home Assistant Core?

Never. New feature

What type of installation are you running?

Home Assistant Container

Integration causing the issue

Tado

Link to integration documentation on our website

No response

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

home-assistant[bot] commented 4 months ago

Hey there @chiefdragon, @erwindouna, mind taking a look at this issue as it has been labeled with an integration (tado) you are listed as a code owner for? Thanks!

Code owner commands Code owners of `tado` can trigger bot actions by commenting: - `@home-assistant close` Closes the issue. - `@home-assistant rename Awesome new title` Renames the issue. - `@home-assistant reopen` Reopen the issue. - `@home-assistant unassign tado` Removes the current integration label and assignees on the issue, add the integration domain after the command. - `@home-assistant add-label needs-more-information` Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue. - `@home-assistant remove-label needs-more-information` Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


tado documentation tado source (message by IssueLinks)

Psythor commented 4 months ago

Hey - this new feature is almost perfect but it's missing some vertical swing options. Though I can now toggle swing on and off (so that it it constantly moving), it's not possible to set the fan to point in a specific direction.

Here's a screenshot from the Tado app showing what options are available as an example:

IMG_4233

erwindouna commented 4 months ago

I'll look into it. Thanks for reporting.

erwindouna commented 4 months ago

@EtienneSOU, I have been looking into it, but I can't reproduce it whereas I don't have an AC of Tado. Do you mind bringing in some support? :)

03397 commented 3 months ago

To shed more light on my issue if I turn off the AC using HVAC mode set to off I do not get the error message I do not know what is the difference of the off from the thermostat with this.

service: climate.set_hvac_mode
target:
  entity_id: climate.living_room
data:
  hvac_mode: "off"
EtienneSOU commented 3 months ago

If you're talking about the error message when turning off the AC, I have a PR waiting reviews. SCR-20240706-qjak

03397 commented 3 months ago

Thanks @EtienneSOU Did you have time to investigate the other issue with the different fan modes missing?

EtienneSOU commented 3 months ago

Hello @03397 The climate component of Home assistant does not have as many options as Tado. That is why we are missing some fan modes. If we want to map every fan modes of tado, we’ll have to add some features to HA climate components and that could affect all AC users, not only Tado users.

03397 commented 3 months ago

Hello @EtienneSOU and thanks for your input in this. I just wanted to let you know that there was a custom component in the past that was mapping correctly all the different fan modes low, low/medium, medium, medium/high, high and silence. This did not affect the HA climate components. How was this possible?

My issue here is that the low fan on HA is indeed low for my system but the remaining too are not. The medium in HA is the low/medium in Tado which is the 2 bars and the high in HA is the medium in Tado which is the 3 bars.

issue-triage-workflows[bot] commented 3 weeks ago

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

03397 commented 3 weeks ago

Nothing was changed.