SiliconLabs / matter

Matter is creating more connections between more objects, simplifying development for manufacturers and increasing compatibility for consumers, guided by the Connectivity Standards Alliance.
https://www.silabs.com/wireless/matter
Apache License 2.0
151 stars 45 forks source link

TC_CC_6.2 and TC_CC_8.1 rate needs to be updated in the Test Plan according to - Rate = (ColorTempPhysicalMaxMireds - ColorTempPhysicalMinMireds)/x and EnhancedHue needs to be updated to 20000 instead of 2000 in TC_CC_8.1 #87

Closed temedi closed 1 year ago

temedi commented 1 year ago

Description of the issue

Rate needs to be updated in the Test Plan according to - Rate = (ColorTempPhysicalMaxMireds - ColorTempPhysicalMinMireds)/x.

In TC's CC_6.2 in commands 2b, 3a, 4a, 4b and CC_8.1 command 4d and EnhancedHue needs to be updated in CC_8.1 command 5a.

Can someone please confirm if the Rate value should be Integer or Float value?

Steps to reproduce the issue

TC_CC_6.2 -
a)  command 2b -
./chip-tool colorcontrol move-color-temperature 1 16319 0 65279 0 0 1 1
Rate = (ColorTempPhysicalMaxMireds - ColorTempPhysicalMinMireds)/40
ColorTempPhysicalMaxMireds = 65279
ColorTempPhysicalMinMireds = 0
Rate = (65279 - 0) / 40 = 1,631.975
Actual Output Rate = 1,631.975
In command Rate mentioned is = 16319

b)  command 3a -
./chip-tool colorcontrol move-color-temperature 3 32369 0 65279 0 0 1 1
Rate = (ColorTempPhysicalMaxMireds - ColorTempPhysicalMinMireds)/20
ColorTempPhysicalMaxMireds = 65279
ColorTempPhysicalMinMireds = 0
Rate = (65279 - 0) / 20 = 3236.95

Actual Output Rate = 3236.95
In command Rate mentioned is = 32369

c)  command 4a -
./chip-tool colorcontrol move-color-temperature 1 32369 0 65279 0 0 1 1
Rate = (ColorTempPhysicalMaxMireds - ColorTempPhysicalMinMireds)/20
ColorTempPhysicalMaxMireds = 65279
ColorTempPhysicalMinMireds = 0
Rate = (65279 - 0) / 20 = 3236.95

Actual Output Rate = 3236.95
In command Rate mentioned is = 32369

d) command 4b - 
./chip-tool colorcontrol move-color-temperature 0 32369 0 65279 0 0 1 1
Rate = (ColorTempPhysicalMaxMireds - ColorTempPhysicalMinMireds)/20
ColorTempPhysicalMaxMireds = 65279
ColorTempPhysicalMinMireds = 0
Rate = (65279 - 0) / 20 = 3236.95

Actual Output Rate = 3236.95
In command Rate mentioned is = 32369

TC_CC_8.1 -
a)  command 4d -
./chip-tool colorcontrol move-color-temperature 1 16319 0 65279 0 0 1 1
Rate = (ColorTempPhysicalMaxMireds - ColorTempPhysicalMinMireds)/40
ColorTempPhysicalMaxMireds = 65279
ColorTempPhysicalMinMireds = 0
Rate = (65279 - 0) / 40 = 1,631.975

Actual Output Rate = 1,631.975
In command Rate mentioned is = 16319

b)  command 5a - EnhancedMoveHue (up) ; start at 20000 with 500 units/s and stopping after every 10s (so increases by 5000 units per 10s)
./chip-tool colorcontrol enhanced-move-to-hue 2000 0 0 0 0 1 1
TH sends EnhancedMoveToHue command to DUT with EnhancedHue=20000, Direction=0x00 (shortest distance) and TransitionTime=0 (immediately).

Device type(s) and network topology

Raspberrypi connected over Wifi

Proposed solution(s)

No response

SMG version

release_0.1.0

Protocol(s)

Wifi

Hardware platform(s)

MG24 ; BRD4186C

Development platform(s)

No response

Captured data or logs

No response

temedi commented 1 year ago

Raised in - https://github.com/project-chip/connectedhomeip/issues/24323