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

[BUG] Incorrect handling of MoveColorTemperature and StepColorTemperature when ColorTemperatureMaximum is 0 #33903

Closed
fuxiaoming-lumi opened this issue Jun 13, 2024 · 0 comments · Fixed by #33983
Assignees
Labels
bug Something isn't working needs triage spec Mismatch between spec and implementation

Comments

@fuxiaoming-lumi
Copy link

Reproduction steps

We have found the issue regarding MoveColorTemperature and StepColorTemperature. If the light receives the above command and ColorTemperatureMaximum is 0, the light will not take any action. The light should adjust its color temperature value according to the spec description.

ColorTemperatureMinimumMireds Field
If the ColorTemperatureMinimumMireds field is set to 0, ColorTempPhysicalMinMireds SHALL 
be used as the lower bound for the ColorTemperatureMireds attribute.

ColorTemperatureMaximumMireds Field
If the ColorTemperatureMaximum Mireds field is set to 0, ColorTempPhysicalMaxMireds 
SHALL be used as the upper bound for the ColorTemperatureMireds attribute.

But the SDK currently does not handle ColorTemperatureMaximum is 0.
https://github.com/project-chip/connectedhomeip/blob/master/src/app/clusters/color-control-server/color-control-server.cpp#L2700
https://github.com/project-chip/connectedhomeip/blob/master/src/app/clusters/color-control-server/color-control-server.cpp#L2811

Bug prevalence

100%

GitHub hash of the SDK that was being used

d83caf7

Platform

core

Platform Version(s)

No response

Anything else?

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs triage spec Mismatch between spec and implementation
Projects
3 participants