Skip to content

Conversation

rzr
Copy link
Collaborator

@rzr rzr commented Sep 29, 2025

Change

Checklist

@rzr rzr force-pushed the phcoval/cc/color/review/main branch from 91d3bfa to 6097a90 Compare September 29, 2025 10:36
rzr and others added 9 commits September 29, 2025 14:35
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>
@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
}

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