-
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
Refresh IM Invoke encoding with latest spec #11169
Refresh IM Invoke encoding with latest spec #11169
Conversation
53662b6
to
9641dcc
Compare
9641dcc
to
5669f1b
Compare
ddeb870
to
ae1e20d
Compare
PR #11169: Size comparison from 1e27d92 to ae1e20d Increases above 0.2%:
Increases (18 builds for efr32, k32w, linux, p6, qpg, telink)
Decreases (8 builds for linux, p6)
Full report (20 builds for efr32, k32w, linux, p6, qpg, telink)
|
ae1e20d
to
90175e0
Compare
PR #11169: Size comparison from 1e27d92 to 90175e0 Increases above 0.2%:
Increases (30 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Decreases (20 builds for esp32, linux, mbed, nrfconnect, p6)
Full report (37 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
9a74b0e
to
d89fc4d
Compare
PR #11169: Size comparison from 1722625 to d89fc4d Increases above 0.2%:
Increases (28 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
Decreases (20 builds for esp32, linux, mbed, nrfconnect, p6)
Full report (34 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
d89fc4d
to
04f017e
Compare
PR #11169: Size comparison from 2fe9024 to 04f017e Increases above 0.2%:
Increases (18 builds for efr32, k32w, linux, p6, qpg, telink)
Decreases (8 builds for linux, p6)
Full report (20 builds for efr32, k32w, linux, p6, qpg, telink)
|
04f017e
to
04b297d
Compare
PR #11169: Size comparison from 12ec702 to 04b297d Increases above 0.2%:
Increases (30 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Decreases (11 builds for linux, mbed, p6)
Full report (37 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Fast tracking, given this has had enough time to get reviews. |
Problem
Refresh IM Invoke encoding with latest spec, and update commandSender and CommandHandler.
Note: Since InvokeRequest is not similar with InvokeRespnse in latest encoding spec, the common logic in Command.cpp and MessageDef has been removed, in follow-up PR, we would remove the Command.cpp and fully split CommandSender and CommandHandler.
https://github.com/CHIP-Specifications/connectedhomeip-spec/blob/master/src/data_model/Encoding-Specification.adoc#encoding-InvokeRequestMessage
Change overview
See above
Testing
All existing Command related test covers.
Add new unit test for InvokeRequest/Response related tlv encoding.