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

Error with lights after installation #2

Closed
ve1koz111 opened this issue May 17, 2023 · 10 comments
Closed

Error with lights after installation #2

ve1koz111 opened this issue May 17, 2023 · 10 comments

Comments

@ve1koz111
Copy link

With Control4 Hass integration by Lawtancool, https://github.com/lawtancool/hass-control4 v1.1.19

All other control 4 devices are working perfectly (especially lights), but media players are not working.

With control4RoomMedia installed, nothing else works except media players, and the following error is presented:

Unexpected error fetching light data: Empty response recieved from Director! The variable LIGHT_LEVEL doesn't seem to exist for any items. 17:39:20 – (ERROR) Control4 (custom integration)

@nalin29
Copy link
Owner

nalin29 commented May 17, 2023

Did it work with the provided HASS version of the control4 integration. If not, this version which will hopefully be merged into main simply builds ontop of the current HASS version and makes no substantive changes to the handling of lights. The control4 integration developed by lawtancool uses websockets but unfortunately no one has datamined the websocket commands to change room controls, only the REST API. Feel free to contribute if you want to try to move the lights and rooms over to websocket.

@ve1koz111
Copy link
Author

ve1koz111 commented May 17, 2023 via email

@nalin29
Copy link
Owner

nalin29 commented May 18, 2023

Not sure, I think if you rename the integration name in the manifest maybe

@nalin29
Copy link
Owner

nalin29 commented May 18, 2023

I have also rebased to the version i have in the HASS pull request as that has been stalled for 2 months now. Hopefully that may fix your issues.

@ve1koz111
Copy link
Author

ve1koz111 commented May 18, 2023

Still getting the same error. Media working, lights not working. When you say you rebased the version, is that the current HA official c4 integration or the one from lawtoncool? I believe this is the same issue from the official integration (home-assistant/core#73353)

Thanks for your help by the way

Logger: custom_components.control4.light
Source: custom_components/control4/director_utils.py:26
Integration: Control4
First occurred: 16:15:19 (1 occurrences)
Last logged: 16:15:19

Unexpected error fetching light data: Empty response recieved from Director! The variable LIGHT_LEVEL doesn't seem to exist for any items.
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 258, in _async_refresh
self.data = await self._async_update_data()
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 217, in _async_update_data
return await self.update_method()
File "/config/custom_components/control4/light.py", line 59, in async_update_data_dimmer
return await update_variables_for_config_entry(
File "/config/custom_components/control4/director_utils.py", line 38, in update_variables_for_config_entry
return await _update_variables_for_config_entry(hass, entry, variable_names)
File "/config/custom_components/control4/director_utils.py", line 26, in _update_variables_for_config_entry
data = await director.getAllItemVariableValue(variable_names)
File "/usr/local/lib/python3.10/site-packages/pyControl4/director.py", line 187, in getAllItemVariableValue
raise ValueError(
ValueError: Empty response recieved from Director! The variable LIGHT_LEVEL doesn't seem to exist for any items.

@nalin29
Copy link
Owner

nalin29 commented May 18, 2023

I did not rebase off of lawtancools custom repo see reasons above. I just brought it up to date with the pending pull request in HASS for the official integration. I think your issue is related to: home-assistant/core#73353

I saw some report of C4 changing the name for light level in the device variables so if you could fire up C4 director and check I might be able to quickly look at a solution if it is that ez. My director is an HC800 so I cannot tests OS 3.3

@nalin29 nalin29 closed this as not planned Won't fix, can't repro, duplicate, stale May 18, 2023
@nalin29 nalin29 reopened this May 18, 2023
@ve1koz111
Copy link
Author

From Eonnet:
image

I believe it is now Brightness Percent. in the pre 3.3.0 this variable was LIGHT_LEVEL in control4.

Let me know if this would be an easy fix for yourself or not.

Thanks

@nalin29
Copy link
Owner

nalin29 commented May 19, 2023

I added a new branch called dev which should try to use the new variable if it is found. Can you test if that works?

@ve1koz111
Copy link
Author

The dev is working perfectly for me, that's fixed the problem.
I would say leave two branches, one for people with OS 3.3+, I will test its stability but I imagine there will be no issues.

Thanks for your help.

@nalin29 nalin29 reopened this May 20, 2023
@nalin29
Copy link
Owner

nalin29 commented May 20, 2023

#3

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