bramstroker / homeassistant-powercalc

Custom component to calculate estimated power consumption of lights and other appliances
MIT License
958 stars 252 forks source link

Add LCX001 #2213

Closed nonamejuju closed 4 months ago

nonamejuju commented 5 months ago

Adding measurement for Philips Hue LCX001

github-actions[bot] commented 5 months ago

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

Image Image

bramstroker commented 5 months ago

Could you check "supported color modes" of your light please. See the questions in the PR template: https://github.com/bramstroker/homeassistant-powercalc/blob/master/.github/PULL_REQUEST_TEMPLATE/power_profile.md

Also please share the manufacturer / model which is in the left top of device page in HA.

nonamejuju commented 5 months ago

Hello, for supported color modes I have:

For the manufacturer/ model I have: "Hue Play gradient lightstrip 55 (929002422702) by Philips"

Please tell me if anything else is needed.

On Sat, Apr 27, 2024, 09:13 Bram Gerritsen @.***> wrote:

Could you check "supported color modes" of your light please. See the questions in the PR template: https://github.com/bramstroker/homeassistant-powercalc/blob/master/.github/PULL_REQUEST_TEMPLATE/power_profile.md

Also please share the manufacturer / model which is in the left top of device page in HA.

— Reply to this email directly, view it on GitHub https://github.com/bramstroker/homeassistant-powercalc/pull/2213#issuecomment-2080396145, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACVJIMP26QJFCFAUQZUU2RDY7NFYZAVCNFSM6AAAAABGRTF3CCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBQGM4TMMJUGU . You are receiving this because you authored the thread.Message ID: @.***>

bramstroker commented 5 months ago

Thanks, that means also measurements for color_temp needs to be provided. As we need to make sure the power profile also works when a user switches the light to color temperature mode.

You can add 929002422702 to aliases in model.json. That will make autodiscovery work correctly.

nonamejuju commented 5 months ago

Thanks. Will do the measurement tomorow.

nonamejuju commented 5 months ago

Uploaded color_temp measurement, should be good now

bramstroker commented 5 months ago

Looking at the visualizations it seems power consumption stays the same starting from brightness level 60. Could you please verify this? Color measurements are hitting the 20W mark, which I would also expect here on max brightness level.

nonamejuju commented 4 months ago

Updated the color_temp measurement this one is hitting the 20W mark. Don't know what happen to previous one 🤷‍♀️

bramstroker commented 4 months ago

@nonamejuju Things are looking fine now. Thanks!

bramstroker commented 4 months ago

Sorry, I want to merge, but I see directory structure is not correct yet.

You have submitted the files in two directories. And the files should be in profile_library now, not custom_components/powercalc/data anymore. I will checkout your PR and fix it.

Screenshot 2024-05-18 at 09 09 05
nonamejuju commented 4 months ago

Thanks for the merge, but I think we forget the alias for the light "929002422702", curently there is no autodiscovery of the lamp.