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

Logs message about turn_on/turn_off methods #17

Open
JackPoint opened this issue Apr 18, 2024 · 2 comments
Open

Logs message about turn_on/turn_off methods #17

JackPoint opened this issue Apr 18, 2024 · 2 comments

Comments

@JackPoint
Copy link

JackPoint commented Apr 18, 2024

Thanks for building this integration. It's really helpful for me to automate in combination with a calendar.

The logs show me the following error with the request to report this as a bug

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

There is no more information. Let me know if I can check or test anything to clarify this.

@JackPoint
Copy link
Author

Looking through the logs I see an error that might be related:

Error while setting up watts_vision platform for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 356, in _async_setup_platform
    await asyncio.shield(awaitable)
  File "/config/custom_components/watts_vision/sensor.py", line 75, in async_setup_entry
    WattsVisionLastCommunicationSensor(
  File "/config/custom_components/watts_vision/central_unit.py", line 16, in __init__
    self._name = "Last communication " + self._label
                 ~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~
TypeError: can only concatenate str (not "NoneType") to str

@pwesters
Copy link
Owner

Not sure they are related.

The first I believe is a change in home assistant. See https://developers.home-assistant.io/blog/2024/01/24/climate-climateentityfeatures-expanded/
I need to update the code

In the second for some reason there’s no value for smarthome. I’ll have to look into that and maybe check for null value.

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

2 participants