-
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
Add IM event python codegen #11802
Add IM event python codegen #11802
Conversation
PR #11802: Size comparison from da558a9 to 93068ac Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
PR #11802: Size comparison from da558a9 to 9758575 Increases (3 builds for mbed)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
9758575
to
8c3cb5c
Compare
PR #11802: Size comparison from 09f9837 to 8c3cb5c Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
8c3cb5c
to
24b619f
Compare
PR #11802: Size comparison from cbd3e95 to 24b619f Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Fast track: python changes only |
return writer.encoding | ||
|
||
@classmethod | ||
def FromTLV(cls, tlvBuffer: bytes): |
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.
Why does this need it's own FromTLV
methods? Why cant it just the versions int he base class?
raise NotImplementedError() | ||
|
||
@ChipUtility.classproperty | ||
def _cluster_object(cls) -> ClusterObject: |
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.
Why is this needed? It was needed for attributes since each attribute was of type ClusterObjectFieldDescriptor
, but in this case, you have a perfectly valid ClusterObjectDescriptor
available here...
Problem
Add event codegen for IM python
Minor fix for GetPriorityLevel and GetEventId in cluster objects
Add ClusterEventDescriptor python class
Change overview
What's in this PR
Testing
Compilation