-
Notifications
You must be signed in to change notification settings - Fork 5
Open
Description
Summary Title:
Improve testing for TC_CC_6_5 , also verify if spec need to be updated.
Description:
During development of TC_CC_6_5 @leorozendaal noticed that there are some inconsistencies during the checks:
Comment: https://github.com/project-chip/connectedhomeip/pull/42413/changes#r2803698075
Comment details:
Looking at the spec, the limits for these attributes are a bit weird.
What I would expect in the attribute table
ColorTempPhysicalMaxMireds: ColorTempPhysicalMinMireds to 65279
ColorTempPhysicalMireds: ColorTempPhysicalMinMireds to ColorTempPhysicalMaxMireds
CoupleColorTempToLevelMinMireds currently has range "ColorTempPhysicalMinMireds to ColorTemperatureMireds" which is weird since I expect this be a rather Fixed value (same for Min and Max) while the stated range depends on the actual CT of the light. I would think ColorTempPhysicalMinMireds to ColorTempPhysicalMaxMireds is more appropriate
(so three F seems to be missing...)
With that, the two conditions in the description of ColorTempPhysicalMinMireds and ColorTempPhysicalMaxMireds could be removed.
TLDR- this is a mess (these spec text bits, not this PR). Don't know how this came about. To be fixed (but not in this PR)
Actions
Verify what is needed to work as expected and update the spec, test and scripts that requiere it.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels
Type
Projects
Status
No status