-
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 Meter Identification cluster #33922
Add Meter Identification cluster #33922
Conversation
PR #33922: Size comparison from 60ae46d to 2d79803 Full report (83 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
@@ -0,0 +1,135 @@ | |||
<?xml version="1.0"?> |
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.
How was this added? data_model files are managed by a script and should be included only as part of script execution (i.e. from parsing asciidoc files)
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.
In the image and likeness of other clusters
05e8621
to
a9463dd
Compare
PR #33922: Size comparison from 6842912 to a9463dd Full report (83 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
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.
Please split this into separate PRs for:
- Add the cluster XML (needs careful review by a set of people)
- Add the cluster implementation (needs careful review by a slightly different set of people)
- Add the example app bits.
@@ -0,0 +1,57 @@ | |||
<?xml version="1.0"?> | |||
<!-- | |||
Copyright (c) 2023 Project CHIP Authors |
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.
This does not look like the right year.
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.
I don't see this cluster in the spec, by the way, which makes reviewing this impossible.
a9463dd
to
abbe8ab
Compare
PR #33922: Size comparison from 667cae8 to abbe8ab Full report (83 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
Fix XML errors
PR #33922: Size comparison from 667cae8 to 6729164 Full report (11 builds for cc32xx, mbed, nrfconnect, qpg, stm32, tizen)
|
PR #33922: Size comparison from 667cae8 to f79c1e1 Increases above 0.2%:
Full report (11 builds for cc32xx, mbed, nrfconnect, qpg, stm32, tizen)
|
PR #33922: Size comparison from 667cae8 to b53abf5 Increases above 0.2%:
Full report (16 builds for cc13x4_26x4, cc32xx, mbed, nrfconnect, qpg, stm32, tizen)
|
b53abf5
to
b663c8e
Compare
PR #33922: Size comparison from 667cae8 to b663c8e Increases above 0.2%:
Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #33922: Size comparison from 667cae8 to bf1ae81 Increases above 0.2%:
Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
Split to 3 PR |
In progress