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

Over-specified ClimateEntityFeature HVACMode(s) #137

Closed
bmos opened this issue Feb 14, 2024 · 8 comments · Fixed by #155
Closed

Over-specified ClimateEntityFeature HVACMode(s) #137

bmos opened this issue Feb 14, 2024 · 8 comments · Fixed by #155

Comments

@bmos
Copy link

bmos commented Feb 14, 2024

Logger: homeassistant.components.climate
Source: components/climate/init.py:354
Integration: Climate (documentation, issues)
First occurred: 9:33:20 AM (1 occurrences)
Last logged: 9:33:20 AM

Entity None (<class 'custom_components.kumo.climate.KumoThermostat'>) implements HVACMode(s): off, cool, dry, heat, fan_only and therefore implicitly supports the turn_on/turn_off methods without setting the proper ClimateEntityFeature. Please create a bug report at https://github.com/dlarrick/hass-kumo/issues

I think this relates to #130

@bmos bmos changed the title Over-specified ClimateEntityFeature Over-specified ClimateEntityFeature HVACMode(s) Feb 14, 2024
@bmos
Copy link
Author

bmos commented Jul 15, 2024

I think this will be resolved by #155

@mhollerb
Copy link

@bmos
I recently had a Kumo unit installed and get the same '...ClimateEntityFeature...' error with a note to submit a bug report. Sounds like this may soon be resolved this, but do you still want a bug report?

Thanks.

@bryanyork
Copy link

I now get entity does not support this service. Is this expected? I am on the latest version. I can longer turn off my Kumo's via Google.

@bryanyork
Copy link

Changing _enable_turn_on_off_backwards_compatibility to True got things working again.

@bryanyork
Copy link

Looks like we need to enable ClimateEntityFeature.TURN_OFF now. I can submit a PR if needed

@bmos
Copy link
Author

bmos commented Aug 1, 2024

I am able to turn mine off and on via HA with all the latest updates.

@bryanyork
Copy link

@bmos the issue is turning off via Google Assistant which expects to issue a climate.turn_off service call to the entity. I could also turn off directly from Home Assistant.

@bmos
Copy link
Author

bmos commented Aug 1, 2024

Ah, got it. So perhaps the issue is with the Google Assistant integration?
Because if you re-add this then it will cause a deprecation warning again I think

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

Successfully merging a pull request may close this issue.

3 participants