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

Add custom profile for Centris 2-spot ceiling light (signify 5061030P7) #2833

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

blixtdevelopment
Copy link

Device information

Added support for the Philips Centris 2-spot ceiling light. Also updated Aliases for LCG002 as the two bundled bulbs are named 5061030P7_02 and 5061030P7_03.

Home Assistant Device information

Hue Centris ceiling (5061030P7_01)
af Signify Netherlands B.V.
Firmware: 1.122.2

Checklist

  • I have created a single PR per device. When you have multiple submissions please create separate PR's.
  • For lights - I have only included the gzipped files (*.gz), not the raw CSV files.
  • For lights - I have provided a CSV file per supported color mode. Look that up in Developer Tools -> States

Additional info

The script saved a standby power of 0.0, but i manually changed it to 0.4 which is in line with similar devices.

Copy link
Contributor

github-actions bot commented Dec 21, 2024

Thank you for submitting light measurements!
Here are some visualizations for the CSV files.

Image
Image

@bramstroker
Copy link
Owner

@blixtdevelopment Thanks for the PR!

Measurements look allright in the graph, but on further inspection I noticed the minimal power consumption as measured is close to or 0.
As also seen when inspecting the file contents:

Screenshot 2024-12-21 at 13 46 37

This can't be correct as these lights should can never consume less than standby power when turned on, which is typically +0.3W for Hue lights.
Did you use a dummy load? That might be the problem when that one was not stable enough.

@blixtdevelopment
Copy link
Author

Hi @bramstroker, yes I did use a dummy load which was turned on for about 1 hour before the script started. I noticed the abnormal small energy consumption in the shelly app when the brightness was low as well.

@bramstroker
Copy link
Owner

Did you also try without dummy load?

@blixtdevelopment
Copy link
Author

Yes, I did that initially. But almost all readings with brightness of 1 it calculated to 0.0W so i stopped it and added the dummy load.

@bramstroker
Copy link
Owner

Hmm ok, what did you use as dummy load?
There is a PR which should make working with a dummy load much more robust: #2810.
But it's not released yet. You might try checking out that branch and running not with docker, but natively on your system . https://github.com/bramstroker/homeassistant-powercalc/blob/master/utils/measure/README.md

@RubenKelevra
Copy link
Contributor

@bramstroker well, I haven't even tested the latest commits. Maybe we should wait for a merge before we recommend users to take measurements ;)

@bramstroker
Copy link
Owner

@bramstroker well, I haven't even tested the latest commits. Maybe we should wait for a merge before we recommend users to take measurements ;)

I agree, but on the other hand it would be good to have feedback from other users/installation with this alpha/beta code.

@RubenKelevra
Copy link
Contributor

I agree, but on the other hand it would be good to have feedback from other users/installation with this alpha/beta code.

I can assure you: It's not running. ;)

If you want people to check it out, refer them to the original commit without the latest changes, or wait for my testing - that will take 1-2 hours. :)

@bramstroker bramstroker mentioned this pull request Jan 15, 2025
3 tasks
@bramstroker
Copy link
Owner

@blixtdevelopment See #2961
Could you tell me more about which entities you have available in HA and which integration you are using to integrate the Hue lights? official hue with hue bridge or Zigbee2MQTT for example?

@blixtdevelopment
Copy link
Author

@bramstroker
I use the official Philips Hue integration with Hue bridge in HA. But i did the measurement script with direct connection to the Hue bridge. The three lights (2 spots, 1 panel) in my Centris are all shown as individual lights in HA.
image

@bramstroker
Copy link
Owner

bramstroker commented Jan 15, 2025

@bramstroker I use the official Philips Hue integration with Hue bridge in HA. But i did the measurement script with direct connection to the Hue bridge. The three lights (2 spots, 1 panel) in my Centris are all shown as individual lights in HA. image

Thanks for quick answer @blixtdevelopment.
So when I understand correctly you controlled the light in measurement with Hue bridge and than it is one light, so you control both spots plus the panel at the same time?
As this is true this imposes a problem as we also have with centris 3 and 4.
Because powercalc discovered and binds these profiles per entity.
That means when the light is at full brightness all 3 power sensors will show 12W, making it a total of 36W.
This is not correct as it should be 12W in total for all of the 3 entities.

edit
On further inspection I see you referenced _02 and _03 to the LCG002 spot, which is actually perfect, which you also mentioned in the PR description doh :-)
When the profile you commited and measured is only the panel it is actually correct, and no need to panic from my side ;-)

@AJediIAm
Copy link
Contributor

AJediIAm commented Jan 15, 2025

FYI: I'm working on some measurements for the Centris 2, Centris 3 and Centris 4. The information will be documented here: #2961 (comment)

The goal is not to create new profiles but to prove (or disprove) that the spots of all Centrix models are the same (and can use Aliases) while the bars are different (and all need to have their own profile).

Thanks @blixtdevelopment for doing the full profile on the Centris 2. Many hands make light work (Dutch saying) ;-)

@blixtdevelopment
Copy link
Author

@bramstroker I only measured the panel that is correct - I have not measured the spots, so that should be all good.

@AJediIAm Glad that you are helping out with this as well! There were some doubts about my measurements for the Centris 2 panel, so it would be awesome if you could validate/disprove my measurement.

@RubenKelevra
Copy link
Contributor

@blixtdevelopment btw you should be able to use #2810 now, as @bramstroker suggested. We worked out most kinks in the meantime.

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

Successfully merging this pull request may close these issues.

4 participants