Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

After upgrade to 2024.8.2, unable to use climate.turn_off or turn_on #164

Open
OmenWild opened this issue Aug 22, 2024 · 9 comments
Open

Comments

@OmenWild
Copy link

I have automations with:

- service: climate.turn_off
  entity_id: climate.computer_room

But after upgrading to 2024.8.2 I get the following error in the log:
Entity climate.computer_room does not support this service.

Additionally, when I try to use Climate: Turn off from Developer Tools -> Actions, it shows No matching entities found when I try to add an entity. Climate: Turn on shows the same lack of entites, but Climate: Set HVAC mode shows all my minisplits, matching what I am seeing in the automations.

My current workaround is this:

- action: climate.set_hvac_mode
  target:
    entity_id: climate.computer_room
  data:
    hvac_mode: "off"

I am currently running the latest released version of this plugin installed with HACS.

@VeniceNerd
Copy link

I use the "change mode" and "set to off" settings. That seems to work.

@ilndboy
Copy link

ilndboy commented Oct 28, 2024

Using 'set hvac mode' to 'off' is not working for me. I have a house full of minisplits that I have to turn off manually. Do we know if there is a fix being worked on for this?

@dlarrick
Copy link
Owner

dlarrick commented Nov 2, 2024

OK, I finally had some time to research this issue. Here's what I found:

https://developers.home-assistant.io/blog/2024/01/24/climate-climateentityfeatures-expanded/

TL;DR: If an integration supports climate.turn_on / climate.turn_off it needs to set the proper ClimateEntityFeature flags, or a warning is emitted. Pull request #155 fixed this deprecation warning, but did so in a way that disables turn_on / turn_off.

If one of you who sees this issue could test this out by adding ClimateEntityFeature.TURN_ON and ClimateEntityFeature.TURN_OFF (delimited by |) to https://github.com/dlarrick/hass-kumo/blob/master/custom_components/kumo/climate.py#L158 in your local copy and try it out, we can see if that fixes the issue and also still avoids the deprecation warning. Or @KTibow if you have some cycles to try this, please do.

@ilndboy
Copy link

ilndboy commented Nov 4, 2024

Thank you for looking into this. I added the code as you instructed and restarted home assistant after doing so. Both my Node Red automation to turn off the heat and the back-up native HA automation I built when I started having this issue are still not turning off the heat when instructed. Both look as though they successfully make the call to turn it off, but when I check the heat is still on.
In Node Red I am trying first trying 'climate.set_hvac_mode' and setting mode to 'off', then delaying a second, then calling 'climate.turn_off' and neither are working.
Let me know if I can help further at all - and thanks again.

@dlarrick
Copy link
Owner

dlarrick commented Nov 5, 2024

The change in this PR: https://github.com/dlarrick/hass-kumo/pull/163/files is what you tried? And it's not working for you? That user reported success.

Just for a sanity check, is your main HomeAssistant version reasonably up to date?

If this doesn't work the next step is to look at what the other (officially-supported) climate integrations are doing here. I doubt I'll have time before the weekend.

@ilndboy
Copy link

ilndboy commented Nov 6, 2024

Following your instructions I interpreted them in this way:
image
This did not work, but the Kumo integration loaded (I could manually control my heat pumps using the climate cards and see the data).

After reading the post you referenced, I changed it to look like this:
image
This caused the Kumo integration to not even be able to load correctly (everything was showing as 'unavailable').

@dlarrick
Copy link
Owner

dlarrick commented Nov 6, 2024

Your earlier one was more correct I think. The 2nd attempt is missing TURN_ON (and has FAN_MODE twice but that should not matter). So let me ask again, what version of HomeAssistant are you running?

@ilndboy
Copy link

ilndboy commented Nov 6, 2024

My apologies - I forgot to answer the other questions;
I always update home assistant and all of my integrations/add ons as soon as updates are available - I can't stand seeing the notification icons! I only do stable builds for everything though - no betas.
I edited my file back to the first version and continue monitoring it.
Thanks again for your work here - most of the reason I was willing to go with these heat pumps is because I knew I could use this integration to control them via Home Assistant. I am grateful for your work here.

@mxr
Copy link
Contributor

mxr commented Dec 13, 2024

FWIW the integration won't load for the second one because you are missing a | at the end of line 158. (But you can remove line 158 altogether too)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants