-
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
[Python] Support reading mixed attributes and events #16265
[Python] Support reading mixed attributes and events #16265
Conversation
PR #16265: Size comparison from 046100b to 5998192 Full report (18 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
5998192
to
331a45e
Compare
PR #16265: Size comparison from 046100b to 331a45e Full report (18 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
331a45e
to
a24aa85
Compare
PR #16265: Size comparison from 4a71f5a to a24aa85 Increases (27 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
Full report (27 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
a24aa85
to
fe33b6f
Compare
PR #16265: Size comparison from cd08938 to fe33b6f Increases (27 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
Full report (27 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
fe33b6f
to
878b09e
Compare
PR #16265: Size comparison from a20ba1c to 878b09e Full report (29 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
878b09e
to
61b25f5
Compare
PR #16265: Size comparison from b58b7be to 61b25f5 Full report (31 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
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.
Fast tracking, given this has had enough time for review (> 3 days)
Problem
We can (and spec allows to) read both attributes and events in a same transaction.
Change overview
Modify the API to expost realted inerface.
Testing
TestMixedReadAttributeAndEvents