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

[deconz] dimmable light groups have no brightness attribute #115633

Closed
Mark-Niemeyer opened this issue Apr 15, 2024 · 6 comments · Fixed by #117588
Closed

[deconz] dimmable light groups have no brightness attribute #115633

Mark-Niemeyer opened this issue Apr 15, 2024 · 6 comments · Fixed by #117588
Assignees

Comments

@Mark-Niemeyer
Copy link

The problem

I have some light groups in deconz which only contain dimmable lights. For these groups, no brightness attribute is available in home assistant. For groups with temperature or colour, the brightness attribute is reported correctly.

json of the dimmable light group from the REST-API:

{
	"action": {
		"alert": "none",
		"bri": 255,
		"colormode": "hs",
		"ct": 0,
		"effect": "none",
		"hue": 0,
		"on": true,
		"sat": 127,
		"scene": null,
		"xy": [
			0,
			0
		]
	},
	"devicemembership": [],
	"etag": "d818412ae47f1fa502430d6d7668329b",
	"id": "4",
	"lights": [
		"22",
		"8"
	],
	"name": "Kueche",
	"scenes": [],
	"state": {
		"all_on": true,
		"any_on": true
	},
	"type": "LightGroup"
}

state of the dimmable light group in home assistant:

entity: light.kueche
state: on
attributes:
  effect_list:
    - colorloop
  supported_color_modes:
    - brightness
  effect: null
  color_mode: unknown
  brightness: null
  is_deconz_group: true
  all_on: true
  friendly_name: Kueche
  supported_features: 44

What version of Home Assistant Core has the issue?

core-2024.4.3

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant Container

Integration causing the issue

deconz

Link to integration documentation on our website

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

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

@home-assistant
Copy link

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

Code owner commands

Code owners of deconz 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 deconz 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)


deconz documentation
deconz source
(message by IssueLinks)

@Kane610
Copy link
Member

Kane610 commented Apr 23, 2024

Thanks for reporting, I think this will take some work to solve this, need to think about how that it can be done

@hans-vd-k
Copy link

Similar issue here. Any progress on solving the issue?

@Kane610
Copy link
Member

Kane610 commented Apr 24, 2024

I probably need to rewrite the whole light platform, no ETA

@JorisKingma
Copy link

Same issue here. Any workaround?

@Kane610
Copy link
Member

Kane610 commented May 16, 2024

Potential fix is up, hopefully will be part of 2024.5.4 release tomorrow

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants