core icon indicating copy to clipboard operation
core copied to clipboard

Hue light intermittently reports incorrect supported_color_modes

Open Mariusthvdb opened this issue 6 months ago • 2 comments

The problem

discovered by using custom integration Powercalc, and because the error is mentioning that integration as logger, I reported there: https://github.com/bramstroker/homeassistant-powercalc/issues/1975

However, author mentions this:

Hi Marius,

light.dorm_ceiling_panel: Lookup table not found (model: LTC016, color_mode: brightness)

This means the light was actually on color_mode brightness at a particular point in time, Powercalc than tries to load the corresponding LUT file, but it can't because there is no available for brightness mode. It should never be in that color mode, because it's also not in the supported_color_modes of the light:

supported_color_modes:

  • color_temp This is a bug in the Hue integration. I think I noticed Marcel about that some time ago, but I don't think anything has been done about that.

You might create another bug report on HA core tracker to report this issue, and get traction on getting this solved.

What version of Home Assistant Core has the issue?

2024.1.0

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Philips Hue

Link to integration documentation on our website

https://www.home-assistant.io/integrations/hue/

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

Deze fout is ontstaan door een aangepaste integratie.

Logger: custom_components.powercalc.strategy.lut
Source: custom_components/powercalc/strategy/lut.py:137
Integration: Powercalc (documentation, issues)
First occurred: 4 januari 2024 om 22:44:37 (2 occurrences)
Last logged: 4 januari 2024 om 22:57:01

light.dorm_ceiling_panel: Lookup table not found (model: LTC016, color_mode: brightness)

Additional information

dev tools info:

min_color_temp_kelvin: 2202
max_color_temp_kelvin: 6535
min_mireds: 153
max_mireds: 454
supported_color_modes:
  - color_temp
color_mode: null
brightness: null
color_temp_kelvin: null
color_temp: null
hs_color: null
rgb_color: null
xy_color: null
mode: normal
dynamics: none
icon: hue:ceiling-round
friendly_name: Dorm ceiling panel
supported_features: 40
Scherm­afbeelding 2024-01-05 om 15 37 56

Mariusthvdb avatar Jan 06 '24 12:01 Mariusthvdb

Hey there @balloob, @marcelveldt, mind taking a look at this issue as it has been labeled with an integration (hue) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of hue can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign hue Removes the current integration label and assignees on the issue, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


hue documentation hue source (message by IssueLinks)

home-assistant[bot] avatar Jan 06 '24 12:01 home-assistant[bot]

Also saw this developer blog post: https://developers.home-assistant.io/blog/2024/02/12/light-color-mode-mandatory

color mode validation is getting more strict (finally). So the scenario reported in the issue will also cause the core warning, and stop the light to work in the future.

bramstroker avatar Feb 17 '24 08:02 bramstroker

This issue should have been fixed a while ago, just forgot to update here. There is one more issue remaining for grouped lights but that one is tracked in a different issue.

marcelveldt avatar Mar 11 '24 11:03 marcelveldt