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

philips: set colorTempRange for LTW013 #5272

Merged
merged 3 commits into from
Jan 6, 2023

Conversation

zegl
Copy link
Contributor

@zegl zegl commented Jan 5, 2023

@sjorge
Copy link
Sponsor Contributor

sjorge commented Jan 5, 2023

Is this actually what the device reports as supported? All hue lights i got my hands on report the same value when reading the color cluster.

You can check these by reading lightingColorCtrl.colorTempPhysicalMin and lightingColorCtrl.colorTempPhysicalMax.

Edit:

I get the same result for my LTG002:

2023-01-05 15:09:08Read result of 'lightingColorCtrl': {"colorTempPhysicalMin":153,"colorTempPhysicalMax":454}

Seems reasonable LWTR013 would have a similar range. Can you also update the range for LTG002? If not I can make a separate PR after this one.

@zegl
Copy link
Contributor Author

zegl commented Jan 5, 2023

@sjorge I didn't check with a physical device, as I don't have one, but the claims made in the original issue seems to be true. I double checked it against some documentation that I found online, which claims a temperature range between 2200 and 6500 K.

@zegl
Copy link
Contributor Author

zegl commented Jan 5, 2023

I'll update the PR to also include the LTG002.

@sjorge
Copy link
Sponsor Contributor

sjorge commented Jan 5, 2023

Ideally we verify it on the actual hardware but since there is usually little variance with Hue and my ws also returns the same range/have matching info on the site. It’s indeed probably OK to do both.

@Koenkk Koenkk merged commit f39494c into Koenkk:master Jan 6, 2023
@Koenkk
Copy link
Owner

Koenkk commented Jan 6, 2023

Thanks!

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

Successfully merging this pull request may close these issues.

Hue white ambiance incorrect colour temperature range
3 participants