-
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
[IM] Wildcard read and chunking #11304
Conversation
245bccb
to
8ae4393
Compare
8ae4393
to
fd26a3f
Compare
PR #11304: Size comparison from ee7682d to fd26a3f Increases above 0.2%:
Increases (27 builds for efr32, k32w, linux, nrfconnect, p6, qpg, telink)
Decreases (9 builds for linux, p6)
Full report (31 builds for efr32, k32w, linux, nrfconnect, p6, qpg, telink)
|
fd26a3f
to
1cfc55d
Compare
Need to make it so that declarations of things that don't depend on generated files are not intermixed in af.h withconnectedhomeip/src/app/AttributePathExpandIterator.cpp Lines 34 to 44 in 1cfc55d
This comment was generated by todo based on a
|
The ClusterInfo may support a group id, the iterator should be able to call group data provider to expand the group id.connectedhomeip/src/app/AttributePathExpandIterator.h Lines 67 to 77 in 1cfc55d
This comment was generated by todo based on a
|
Optimize this implementation by making the iterator only emit intersected paths.connectedhomeip/src/app/reporting/Engine.cpp Lines 111 to 121 in 1cfc55d
This comment was generated by todo based on a
|
1cfc55d
to
4aa62af
Compare
Need to make it so that declarations of things that don't depend on generated files are not intermixed in af.h withconnectedhomeip/src/app/AttributePathExpandIterator.cpp Lines 34 to 44 in 4aa62af
This comment was generated by todo based on a
|
014b556
to
afae79c
Compare
PR #11304: Size comparison from 507ba15 to afae79c Increases above 0.2%:
Increases (25 builds for efr32, esp32, k32w, linux, mbed, p6, qpg, telink)
Decreases (5 builds for mbed, p6)
Full report (28 builds for efr32, esp32, k32w, linux, mbed, p6, qpg, telink)
|
afae79c
to
d419026
Compare
PR #11304: Size comparison from 5623978 to d419026 Increases above 0.2%:
Increases (25 builds for efr32, esp32, k32w, linux, mbed, p6, qpg, telink)
Decreases (5 builds for mbed, p6)
Full report (28 builds for efr32, esp32, k32w, linux, mbed, p6, qpg, telink)
|
448e6fc
to
527cd7d
Compare
PR #11304: Size comparison from 5623978 to 527cd7d Increases above 0.2%:
Increases (34 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Decreases (5 builds for mbed, p6)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
b31e904
to
6924534
Compare
PR #11304: Size comparison from fcf4c33 to 6924534 Increases above 0.2%:
Increases (32 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Decreases (5 builds for mbed, p6)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
93d5c99
to
6cdcecc
Compare
PR #11304: Size comparison from e755a26 to 6cdcecc Increases above 0.2%:
Increases (15 builds for esp32, mbed, nrfconnect, qpg, telink)
Decreases (3 builds for mbed)
Full report (21 builds for esp32, mbed, nrfconnect, qpg, telink)
|
Wildcard cluster id with non-global attributes or wildcard attribute paths should be rejected.connectedhomeip/src/app/ReadHandler.cpp Lines 357 to 367 in 2c50c6c
This comment was generated by todo based on a
|
2c50c6c
to
9f86049
Compare
PR #11304: Size comparison from cbd3e95 to 9f86049 Increases above 0.2%:
Increases (17 builds for efr32, esp32, k32w, mbed, p6, qpg, telink)
Decreases (5 builds for mbed, p6)
Full report (20 builds for efr32, esp32, k32w, mbed, p6, qpg, telink)
|
9f86049
to
2627266
Compare
PR #11304: Size comparison from cbd3e95 to 2627266 Increases above 0.2%:
Increases (32 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Decreases (5 builds for mbed, p6)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
fast track: very old PR, created and reviewed by a domain owner. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks pretty reasonable, fwiw. Various typos in comments, but...
* [IM] Support chunking in report engine * Address comments * Support wildcard path in read / subscribe interaction
Problem
Change overview
Testing
TestReadWildcard
andTestReadChunking
for test these featuresTestReadChunking
reads several large mocked attributes to trigger chunking features.