-
Notifications
You must be signed in to change notification settings - Fork 22
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
Deprecated constant, ClimateEntityFeature warning #19
Comments
You're welcome to apply these changes to get rid of warnings. It also includes a new functionality, but it shouldn't be hard to isolate only deprecation fixes. |
Thanks, @lumixen, for the PR. It's been merged and released. |
Thanks for your contribution, however with the updated version the terneo entity turns to unavailable after a while and never recovers. Reverting back to previous version fixes this, no idea what needs to be done differently. |
@sitavata thanks for reporting! Could you please try to power off / power on the thermostat itself with the updated version and check what happens? I've recently experienced a similar behavior on my terneo sx devices. After a couple days of runtime all terneo devices started provide inconsistent data and one of them even stopped responding to API requests and therefore went to unavailable state. Looks very similar to this issue. Before this change climate device was reporting last captured state, but now, if it's not properly responding to API requests, it goes to unavailable state (which actually makes sense). |
Well now that you mention API requests then I have constantly had error messages in system logs related to Terneo which contains the following message (but keep in mind that this has not caused any noticeable issues with operation of the thermostat in my case): HTTPConnectionPool: Max retries exceeded with url: /api.cgi (Caused by ConnectTimeoutError(<urllib3.connection.HTTPConnection object at 0xffff83caa990>, 'Connection to IP address timed out. (connect timeout=5)')) I will update once more and cycle power when this happens, will let you know! |
I can tell you that the same error caused the entity to become unavailable and a power cycle to the thermostat fixed it. |
Thanks for investigation! If it automatically recovers after power cycle - it means that the home assistant component itself is functioning properly and constantly polls the device even if it becomes unavailable, but does not get the correct response most of the time. And that leads me to the thought that issue we're observing is caused by some sort of firmware instability that makes device to respond irregularly and eventually completely stop responding after a while. I see a very similar behavior on my setup approximately after a 24-48 hours (I have 4 SX devices sitting on firmware 2.4/2.5). For now I have had to fallback to terneo cloud integration from @ishikht as I couldn't get my devices to have a consistent behavior using their local API, they do break eventually and the only way to fix it seems to be power reset. |
TEMP_CELSIUS was used from terneo, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfTemperature.CELSIUS instead
Entity None (<class 'custom_components.terneo.climate.ThermostatDevice'>) implements HVACMode(s): auto, heat, off and therefore implicitly supports the turn_on/turn_off methods without setting the proper ClimateEntityFeature
The text was updated successfully, but these errors were encountered: