-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[Feature]Add operational state cluster xml and code gen #26736
[Feature]Add operational state cluster xml and code gen #26736
Conversation
PR #26736: Size comparison from 853691d to 8da17c8 Increases above 0.2%:
Increases (23 builds for bl702, cc32xx, esp32, linux, psoc6, telink)
Decreases (6 builds for efr32, telink)
Full report (57 builds for bl602, bl702, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
src/app/zap-templates/zcl/data-model/chip/operational-state-cluster.xml
Outdated
Show resolved
Hide resolved
src/app/zap-templates/zcl/data-model/chip/operational-state-cluster.xml
Outdated
Show resolved
Hide resolved
src/app/zap-templates/zcl/data-model/chip/operational-state-cluster.xml
Outdated
Show resolved
Hide resolved
src/app/zap-templates/zcl/data-model/chip/operational-state-cluster.xml
Outdated
Show resolved
Hide resolved
PR #26736: Size comparison from 014166c to aaca4de Increases above 0.2%:
Increases (33 builds for bl602, bl702, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
Decreases (7 builds for bl702, telink)
Full report (57 builds for bl602, bl702, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
@mideayanghui There seem to be java codegen bits missing? |
PR #26736: Size comparison from d34a250 to da01f35 Increases above 0.2%:
Increases (25 builds for bl602, bl702, cc32xx, esp32, linux, psoc6, qpg, telink)
Decreases (10 builds for bl702, cyw30739, efr32, esp32, qpg, telink)
Full report (57 builds for bl602, bl702, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
Do you have any suggestions for fix this bug of java codegen? Thanks. |
It's possible that running regen_all twice will fix things. @andy31415 |
PR #26736: Size comparison from bf70b9e to 56e6a94 Increases above 0.2%:
Increases (26 builds for bl602, bl702, cc32xx, esp32, linux, psoc6, telink)
Decreases (10 builds for efr32, nrfconnect, qpg, telink)
Full report (58 builds for bl602, bl702, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
PR #26736: Size comparison from 4b04598 to c80938c Increases above 0.2%:
Increases (23 builds for bl702, cc13x4_26x4, cc32xx, cyw30739, esp32, linux, nrfconnect, psoc6, telink)
Decreases (13 builds for bl602, cyw30739, efr32, esp32, k32w, nrfconnect, telink)
Full report (66 builds for bl602, bl702, cc13x2_26x2, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
PR #26736: Size comparison from adef33d to 7f36b17 Increases above 0.2%:
Increases (23 builds for bl602, bl702, cc13x4_26x4, cc32xx, esp32, linux, psoc6, telink)
Decreases (14 builds for bl702, cc13x4_26x4, efr32, esp32, nrfconnect, telink)
Full report (66 builds for bl602, bl702, cc13x2_26x2, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
Seperate the operational state cluster XML to a new PR. Based on
[New XML and Code Gen for Appliance Clusters (OpState, TCC, RefAlm #26560 )](https://github.com/project-chip/connectedhomeip/pull/26560#top)
Problem:
Matter 1.2 clusters missing XML and generated code for operational state cluster
Solution:
Create XML
Update zcl.json, zcl-with-test-extensions.json and related zap files.
Testing:
Regen all
Rebuild chip tool
Verify all new clusters are visible in chip-tool, with correct set of available attributes