Skip to content

Conversation

@rzr
Copy link
Collaborator

@rzr rzr commented Sep 29, 2025

Change

Checklist

rzr added a commit that referenced this pull request Sep 29, 2025
This reverts commit 8d284db.

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>
rzr added a commit that referenced this pull request Sep 29, 2025
Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>
@rzr 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 Sep 29, 2025
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 Sep 29, 2025
Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>
rzr pushed a commit that referenced this pull request Sep 29, 2025
Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>
rzr pushed a commit that referenced this pull request Sep 29, 2025
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 Sep 29, 2025
Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>
rzr pushed a commit that referenced this pull request Sep 29, 2025
Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>
rzr pushed a commit that referenced this pull request Sep 29, 2025
Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>
@DucAnh-silabs
Copy link

DucAnh-silabs commented Oct 1, 2025

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 and others added 9 commits October 7, 2025 15:06
This reverts commit 8d284db.

Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>
Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>
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>
Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>
Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>
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>
Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>
Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>
Origin: #146
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>
@rzr rzr force-pushed the phcoval/cc/color/review/main branch from 6097a90 to e202f3e Compare October 7, 2025 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants