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

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 opened this issue Jan 9, 2023 · 1 comment
Assignees
Labels
wont-fix Silabs determined this issue will not be resolved by Silabs

Comments

@temedi
Copy link

temedi commented Jan 9, 2023

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

@github-actions github-actions bot added the triage The issue is being processed label Jan 9, 2023
@temedi temedi closed this as completed Jan 9, 2023
@temedi
Copy link
Author

temedi commented Jan 9, 2023

Raised in - project-chip#24323

@silabs-sebastien silabs-sebastien added wont-fix Silabs determined this issue will not be resolved by Silabs and removed triage The issue is being processed labels Jan 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wont-fix Silabs determined this issue will not be resolved by Silabs
Projects
None yet
Development

No branches or pull requests

2 participants