-
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
Fixed All Clusters App ZAP Test Script File #25045
Fixed All Clusters App ZAP Test Script File #25045
Conversation
emargolis
commented
Feb 14, 2023
- Removed all Thread Network Diagnostics Cluster events, which are currently not implemented in the SDK at all.
- Removed all Actions Cluster events, which are currently not implemented in the SDK.
- Removed BatFaultChange event of the Power Source Cluster, which is not implemented in the SDK..
- Removed StateChange event of the Boolean State Cluster, which is not implemented for the All Clusters App.
- Removed all events of the Pump Configuration and Control Cluster, which are currently not implemented in the SDK.
- Removed all Thread Network Diagnostics Cluster events, which are currently not implemented in the SDK at all. - Removed all Actions Cluster events, which are currently not implemented in the SDK. - Removed BatFaultChange event of the Power Source Cluster, which is not implemented in the SDK.. - Removed StateChange event of the Boolean State Cluster, which is not implemented for the All Clusters App. - Removed all events of the Pump Configuration and Control Cluster, which are currently not implemented in the SDK.
@emargolis could you update this and #25046 , #25047 to master since it includes #24991? I hope that event list changes will start showing up in matter files. |
@emargolis why do we need to change the test file? The test file is not building any apps and just ensures we know what deltas occur when we alter zapt or zap versions. |
Yes, I agree that there is no strong reason to update this test script but still wanted to have it aligned with the actual example zap file. |
PR #25045: Size comparison from 1d6a958 to a5468b5 Increases (6 builds for bl602, cc32xx, psoc6, telink)
Decreases (6 builds for bl702, efr32, psoc6, telink)
Full report (35 builds for bl602, bl702, cc13x2_26x2, cc32xx, cyw30739, efr32, linux, mbed, psoc6, qpg, telink)
|
All mentioned and other PR were updated accordingly |
* Fixed All Clusters App ZAP Test Script File - Removed all Thread Network Diagnostics Cluster events, which are currently not implemented in the SDK at all. - Removed all Actions Cluster events, which are currently not implemented in the SDK. - Removed BatFaultChange event of the Power Source Cluster, which is not implemented in the SDK.. - Removed StateChange event of the Boolean State Cluster, which is not implemented for the All Clusters App. - Removed all events of the Pump Configuration and Control Cluster, which are currently not implemented in the SDK. * ZAP REGEN