-
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
Window Covering: Add FeatureMap #9246
Window Covering: Add FeatureMap #9246
Conversation
96db366
to
a55036a
Compare
@jmeg-sfy Sorry conflicts here :( |
a55036a
to
4218afa
Compare
@woody-apple ToT rebased ! |
4218afa
to
6467f70
Compare
@woody-apple ToT rebased conflict fixed (again) |
726422e
to
257b4e6
Compare
@jmeg-sfy Can you resolve conflicts please? |
You're welcome to merge yourself FYI, once it builds and conflicts are resolved. |
PR #9246: Size comparison from 4b1bc90 to 2996f40 Increases (6 builds for efr32, esp32, linux, mbed)
Decreases (1 build for mbed)
Full report (36 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
PR #9246: Size comparison from 9b8410f to 65dbbf9 Increases (16 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, qpg, telink)
Decreases (3 builds for esp32, linux, mbed)
Full report (36 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
PR #9246: Size comparison from 2149dae to dada6df Full report (8 builds for k32w, p6, qpg, telink)
|
/rebase |
1 similar comment
/rebase |
PR #9246: Size comparison from 1d10ef8 to 3af5516 Increases (6 builds for efr32, esp32, linux, mbed)
Decreases (1 build for mbed)
Full report (36 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Size increase report for "esp32-example-build" from 1d10ef8
Full report output
|
@woody-apple sorry i didnt see your comment earlier |
We had some changes to .xml files to rename things that were not reflected into .zap files. Running ./scripts/tools/zap_convert_all.py fixes that, but fails before this PR because ZAP support for <tag> is broken and project-chip#9246 introduced use of <tag> in window-covering.xml. So the changes: 1) Comment out the <tag> bits in src/app/zap-templates/zcl/data-model/chip/window-covering.xml. 2) Run ./scripts/tools/zap_convert_all.py to fix up the attribute names in the .zap files.
We had some changes to .xml files to rename things that were not reflected into .zap files. Running ./scripts/tools/zap_convert_all.py fixes that, but fails before this PR because ZAP support for <tag> is broken and #9246 introduced use of <tag> in window-covering.xml. So the changes: 1) Comment out the <tag> bits in src/app/zap-templates/zcl/data-model/chip/window-covering.xml. 2) Run ./scripts/tools/zap_convert_all.py to fix up the attribute names in the .zap files.
* WC: Add FeatureMap inside xml + zap * WC: Update all autogenerated files by ZAP + SDK * WC: Activate Test for global attribute FeatureMap reading value * WC: Add missing test dependancies * WC: Add absolute positioning into FeatureMap * Fix: Conflict on Cluster gen * WC: Regenerate/Rebase * WC: Regen / Rebase * WC: Regen rebase * Restyled by autopep8 * Restyled by autopep8 Co-authored-by: Justin Wood <[email protected]> Co-authored-by: Restyled.io <[email protected]>
We had some changes to .xml files to rename things that were not reflected into .zap files. Running ./scripts/tools/zap_convert_all.py fixes that, but fails before this PR because ZAP support for <tag> is broken and project-chip#9246 introduced use of <tag> in window-covering.xml. So the changes: 1) Comment out the <tag> bits in src/app/zap-templates/zcl/data-model/chip/window-covering.xml. 2) Run ./scripts/tools/zap_convert_all.py to fix up the attribute names in the .zap files.
* WC: Add FeatureMap inside xml + zap * WC: Update all autogenerated files by ZAP + SDK * WC: Activate Test for global attribute FeatureMap reading value * WC: Add missing test dependancies * WC: Add absolute positioning into FeatureMap * Fix: Conflict on Cluster gen * WC: Regenerate/Rebase * WC: Regen / Rebase * WC: Regen rebase * Restyled by autopep8 * Restyled by autopep8 Co-authored-by: Justin Wood <[email protected]> Co-authored-by: Restyled.io <[email protected]>
We had some changes to .xml files to rename things that were not reflected into .zap files. Running ./scripts/tools/zap_convert_all.py fixes that, but fails before this PR because ZAP support for <tag> is broken and project-chip#9246 introduced use of <tag> in window-covering.xml. So the changes: 1) Comment out the <tag> bits in src/app/zap-templates/zcl/data-model/chip/window-covering.xml. 2) Run ./scripts/tools/zap_convert_all.py to fix up the attribute names in the .zap files.
* WC: Add FeatureMap inside xml + zap * WC: Update all autogenerated files by ZAP + SDK * WC: Activate Test for global attribute FeatureMap reading value * WC: Add missing test dependancies * WC: Add absolute positioning into FeatureMap * Fix: Conflict on Cluster gen * WC: Regenerate/Rebase * WC: Regen / Rebase * WC: Regen rebase * Restyled by autopep8 * Restyled by autopep8 Co-authored-by: Justin Wood <[email protected]> Co-authored-by: Restyled.io <[email protected]>
We had some changes to .xml files to rename things that were not reflected into .zap files. Running ./scripts/tools/zap_convert_all.py fixes that, but fails before this PR because ZAP support for <tag> is broken and project-chip#9246 introduced use of <tag> in window-covering.xml. So the changes: 1) Comment out the <tag> bits in src/app/zap-templates/zcl/data-model/chip/window-covering.xml. 2) Run ./scripts/tools/zap_convert_all.py to fix up the attribute names in the .zap files.
Change overview
This PR add the missing FeatureMap global attribute:
This is also following this specs PR https://github.com/CHIP-Specifications/connectedhomeip-spec/pull/3794
Testing
How was this tested? (at least one bullet point required)