-
Notifications
You must be signed in to change notification settings - Fork 2k
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
TV Matter Media: Updated existing cluster to sync it with latest spec change #31782
TV Matter Media: Updated existing cluster to sync it with latest spec change #31782
Conversation
PR #31782: Size comparison from d46f426 to 35b3975 Full report (29 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, k32w, mbed, nrfconnect, qpg, stm32)
|
PR #31782: Size comparison from 6fd32ba to 42dd4d2 Full report (9 builds for cc32xx, mbed, nrfconnect, qpg, stm32)
|
@lazarkov - why is Please add details in the summary (I assume this is a close followup on a previous change, so no released versions with the old structures should exist) |
42dd4d2
to
fd2c277
Compare
Added info in the description |
PR #31782: Size comparison from af29ce6 to b365864 Increases above 0.2%:
Increases (5 builds for esp32, linux)
Decreases (1 build for efr32)
Full report (73 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, stm32, telink)
|
PR #31782: Size comparison from 1b87b4a to 92ede74 Increases above 0.2%:
Increases (2 builds for esp32, linux)
Full report (41 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, stm32)
|
PR #31782: Size comparison from 5f955f4 to ae408c9 Increases above 0.2%:
Increases (5 builds for esp32, linux)
Full report (69 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, stm32, telink)
|
PR #31782: Size comparison from 219c3f0 to 0578255 Increases (2 builds for esp32, linux)
Decreases (1 build for linux)
Full report (41 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, stm32)
|
PR #31782: Size comparison from b1bf11e to 0ef533b Increases (5 builds for esp32, linux)
Decreases (2 builds for efr32, linux)
Full report (73 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, stm32, telink)
|
PR #31782: Size comparison from cdd128b to 8e90ee8 Increases (5 builds for esp32, linux)
Decreases (2 builds for efr32, linux)
Full report (73 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, stm32, telink)
|
PR #31782: Size comparison from 058f199 to d014976 Increases (5 builds for esp32, linux)
Decreases (2 builds for efr32, linux)
Full report (73 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, stm32, telink)
|
Problem
The message cluster has been updated under PR https://github.com/CHIP-Specifications/connectedhomeip-spec/pull/8812
Change overview
Skipped Backward Compatibility
This is a quick followup on a PR, so no released versions with the old structures should exist
Tests
Build the TV app using command:
scripts/examples/gn_build_example.sh examples/tv-app/linux out/debug/
Built the CHIP tool test client using command:
scripts/examples/gn_build_example.sh examples/chip-tool out/debug/ chip_config_network_layer_ble=false
Run tests using command:
./scripts/run_in_build_env.sh \ "./scripts/tests/run_test_suite.py \ --runner chip_tool_python \ --chip-tool ./out/debug/chip-tool \ run \ --iterations 1 \ --test-timeout-seconds 120 \ --tv-app ./out/debug/chip-tv-app \ "