Color Switch #146

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.

Sign up for GitHub

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

Jump to bottom

Merged

rzr merged 10 commits into main from phcoval/cc/color/review/main

Oct 28, 2025

Collaborator

rzr commented Sep 29, 2025

Change

Checklist

A Contribution License Agreement has been established between @SiliconLabs and author's company (matching email domain)

rzr added a commit that referenced this pull request


          Revert "NOJIRA: Rename color switch cc to cpp for clang support (#24)"

ef6ca40

This reverts commit 8d284db.

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr added a commit that referenced this pull request


          UIC-3272: Import Unify patches for ColorSwitch

9ae7a5e

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr force-pushed the phcoval/cc/color/review/main branch from 91d3bfa to 6097a90 Compare

September 29, 2025 10:36

rzr pushed a commit that referenced this pull request


          UIC-3272: Switch Color Command Class Cpp

c145d8d

UIC-3272: Fix mismatch attribute size

Due to legacy reason some attributes are stored as uint32_t while being single byte (uint8_t). This cause issue with the C++ API and those attribute needs to manually set a raw bytes.

UIC-3272: fixed issue with types value

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Custom Switch Color cluster

9ec15e2

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Switch Color uam file

571eaf5

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Remove Switch Color rust implementation

2d6178b

This one of the two classes still handled by rust. Since rust is only partially supported, we won't maintain this.

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Add Python Script for CTT test Switch Color Command Class

94b1365

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Documentation for the Color Switch command class.

793555d

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Documentation for the Color Switch command class.

6097a90

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

DucAnh-silabs commented Oct 1, 2025 •

edited

Loading

I've tested the Color Switch Command Class with the following setup:

Setup

EN: BRD2603A (ZGM230S) + Led Bulb (2025.6.1)
Controller: ZGM230S + 2025.6.1 on WSL2 Ubuntu 22.04.5LTS

Tests

Changing level of each Color id {2, 3, 4} between 0 and 255.
Start a Up/Down Level Change for each Color id {2, 3, 4} with different duration {0, 10, 100}

Verification

Successfully change Attribute value
Zniffer shows commands successfully being sent and there are reports with correct state.
Successfully change color on BRD2603A

MQTT topics and payloads

`ucl/by-unid/<UNID>/<EP>/UnifySwitchColor/Commands/SetColor`

{
"ColorComponentId": 2,
"Value": 100,
"Duration": 0
}

`ucl/by-unid/<UNID>/<EP>/UnifySwitchColor/Commands/StartStopChange`

{
"StartStop": true,
"UpDown": false,
"IgnorStartLevel": true,
"ColorComponentId": 2,
"StartLevel": 50,
"Duration": 10
}

rzr added a commit that referenced this pull request


          Revert "NOJIRA: Rename color switch cc to cpp for clang support (#24)"

36f9aa9

This reverts commit 8d284db.

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr force-pushed the phcoval/cc/color/review/main branch from 6097a90 to e202f3e Compare

October 7, 2025 13:06

rzr added a commit that referenced this pull request


          UIC-3272: Import Unify patches for ColorSwitch

1c1e6c4

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Switch Color Command Class Cpp

535c2f2

UIC-3272: Fix mismatch attribute size

Due to legacy reason some attributes are stored as uint32_t while being single byte (uint8_t). This cause issue with the C++ API and those attribute needs to manually set a raw bytes.

UIC-3272: fixed issue with types value

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Custom Switch Color cluster

1dbc091

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Switch Color uam file

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Remove Switch Color rust implementation

afbf3a2

This one of the two classes still handled by rust. Since rust is only partially supported, we won't maintain this.

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Add Python Script for CTT test Switch Color Command Class

8a9161d

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Documentation for the Color Switch command class.

e700241

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Documentation for the Color Switch command class.

e202f3e

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr added a commit that referenced this pull request


          Revert "NOJIRA: Rename color switch cc to cpp for clang support (#24)"

8ab59fd

This reverts commit 8d284db.

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr added a commit that referenced this pull request


          UIC-3272: Import Unify patches for ColorSwitch

9f28787

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Switch Color Command Class Cpp

140f885

UIC-3272: Fix mismatch attribute size

Due to legacy reason some attributes are stored as uint32_t while being single byte (uint8_t). This cause issue with the C++ API and those attribute needs to manually set a raw bytes.

UIC-3272: fixed issue with types value

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Custom Switch Color cluster

79c1634

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Switch Color uam file

51282e7

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr force-pushed the phcoval/cc/color/review/main branch from e202f3e to 96505b4 Compare

October 28, 2025 16:24

rzr pushed a commit that referenced this pull request


          UIC-3272: Remove Switch Color rust implementation

4a9b301

This one of the two classes still handled by rust. Since rust is only partially supported, we won't maintain this.

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Add Python Script for CTT test Switch Color Command Class

5a93c81

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Documentation for the Color Switch command class.

130a300

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Documentation for the Color Switch command class.

96505b4

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr requested review from silabs-JulienT and silabs-aydogane

October 28, 2025 16:34

silabs-benedek approved these changes

View reviewed changes

rzr and others added 10 commits

October 28, 2025 17:52


          fix(ci): Release on tag push

0ed8805

For some reason the step was skip,
I believe a gh update caused that regression

Signed-off-by: Philippe Coval <philippe.coval@silabs.com>


          Revert "NOJIRA: Rename color switch cc to cpp for clang support (#24)"

5d6e3e0

This reverts commit 8d284db.

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>


          UIC-3272: Import Unify patches for ColorSwitch

11ed4a5

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>


          UIC-3272: Switch Color Command Class Cpp

e11122d

UIC-3272: Fix mismatch attribute size

Due to legacy reason some attributes are stored as uint32_t while being single byte (uint8_t). This cause issue with the C++ API and those attribute needs to manually set a raw bytes.

UIC-3272: fixed issue with types value

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>


          UIC-3272: Custom Switch Color cluster

0a9da54

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>


          UIC-3272: Switch Color uam file

69dbf76

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>


          UIC-3272: Remove Switch Color rust implementation

6005b6f

This one of the two classes still handled by rust. Since rust is only partially supported, we won't maintain this.

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>


          UIC-3272: Add Python Script for CTT test Switch Color Command Class

cea8980

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>


          UIC-3272: Documentation for the Color Switch command class.

0fbe892

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>


          UIC-3272: Documentation for the Color Switch command class.

c7730cf

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr force-pushed the phcoval/cc/color/review/main branch from 96505b4 to c7730cf Compare

October 28, 2025 16:52

rzr merged commit 1140eda into main

1 of 2 checks passed

rzr added a commit that referenced this pull request


          Revert "NOJIRA: Rename color switch cc to cpp for clang support (#24)"

bb23aab

This reverts commit 8d284db.

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr added a commit that referenced this pull request


          UIC-3272: Import Unify patches for ColorSwitch

28f9664

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Switch Color Command Class Cpp

d0fd88e

UIC-3272: Fix mismatch attribute size

Due to legacy reason some attributes are stored as uint32_t while being single byte (uint8_t). This cause issue with the C++ API and those attribute needs to manually set a raw bytes.

UIC-3272: fixed issue with types value

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Custom Switch Color cluster

f230cd1

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Switch Color uam file

0a42cf1

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Remove Switch Color rust implementation

909dc17

This one of the two classes still handled by rust. Since rust is only partially supported, we won't maintain this.

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Add Python Script for CTT test Switch Color Command Class

2380b7b

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

rzr pushed a commit that referenced this pull request


          UIC-3272: Documentation for the Color Switch command class.

9f4fc20

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet