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
69.82k stars 28.94k forks source link

Daikin AC integration: set thermostat in 0.5 precision #84816

Closed sidlauskas closed 1 year ago

sidlauskas commented 1 year ago

The problem

I found that with Daikin AC integration the thermostat precision is incorrect (1.0). The precision for Daikin AC should be 0.5

What version of Home Assistant Core has the issue?

core-2022.12.8

What was the last working version of Home Assistant Core?

No response

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?

No response

Additional information

No response

home-assistant[bot] commented 1 year ago

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

Code owner commands Code owners of `daikin` can trigger bot actions by commenting: - `@home-assistant close` Closes the issue. - `@home-assistant rename Awesome new title` Change the title of the issue. - `@home-assistant reopen` Reopen the issue. - `@home-assistant unassign daikin` Removes the current integration label and assignees on the issue, add the integration domain after the command.

(message by CodeOwnersMention)


daikin documentation daikin source (message by IssueLinks)

issue-triage-workflows[bot] commented 1 year 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.

fredrike commented 1 year ago

@sidlauskas this is not true for all models. We have had discussions earlier regarding precision, please search for other Dainin issues and refer to the here and I'll see what we can do.

DragonQ commented 1 year ago

Although the IR controllers support setting temperature in 0.5 degree increments, the official Daikin Online Controller app only supports 1 degree increments, at least for my Stylish models. Given that app is using the same API as this integration, I suspect the API is the limiting factor here, unfortunately.

sidlauskas commented 1 year ago

Although the IR controllers support setting temperature in 0.5 degree increments, the official Daikin Online Controller app only supports 1 degree increments, at least for my Stylish models. Given that app is using the same API as this integration, I suspect the API is the limiting factor here, unfortunately.

Daikin ONECTA app supports 0.5 degree (celsius) increments for my Stylish air conditioner

DragonQ commented 1 year ago

Although the IR controllers support setting temperature in 0.5 degree increments, the official Daikin Online Controller app only supports 1 degree increments, at least for my Stylish models. Given that app is using the same API as this integration, I suspect the API is the limiting factor here, unfortunately.

Daikin ONECTA app supports 0.5 degree (celsius) increments for my Stylish air conditioner

Does the ONECTA app use the same local API though? It's a cloud based app and probably uses their closed cloud API.

fredrike commented 1 year ago

Again, this have been discussed earlier and the conclusion were to stick with integer increments. If there were an easy way to change this setting we could try to implement it.

There is support for options and 0.5 increments could be such a option but I have limited time to implement it https://developers.home-assistant.io/docs/config_entries_options_flow_handler.

fredrike commented 1 year ago

@home-assistant rename Daikin AC integration: set thermostat in 0.5 precision

fredrike commented 1 year ago

@home-assistant close Not planned (insufficient time)