-
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
[Feature]Microwave oven control cluster implementation #30235
Conversation
PR #30235: Size comparison from 16f8d7a to ad8e132 Decreases (1 build for efr32)
Full report (72 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, 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.
This looks much better now!
examples/microwave-oven-app/microwave-oven-common/include/microwave-oven-device.h
Outdated
Show resolved
Hide resolved
examples/microwave-oven-app/microwave-oven-common/include/microwave-oven-device.h
Outdated
Show resolved
Hide resolved
examples/microwave-oven-app/microwave-oven-common/include/microwave-oven-device.h
Outdated
Show resolved
Hide resolved
examples/microwave-oven-app/microwave-oven-common/include/microwave-oven-device.h
Outdated
Show resolved
Hide resolved
src/app/clusters/microwave-oven-control-server/microwave-oven-control-server.cpp
Outdated
Show resolved
Hide resolved
src/app/clusters/microwave-oven-control-server/microwave-oven-control-server.cpp
Outdated
Show resolved
Hide resolved
src/app/clusters/microwave-oven-control-server/microwave-oven-control-server.cpp
Outdated
Show resolved
Hide resolved
src/app/clusters/microwave-oven-control-server/microwave-oven-control-server.cpp
Outdated
Show resolved
Hide resolved
src/app/clusters/microwave-oven-control-server/microwave-oven-control-server.cpp
Outdated
Show resolved
Hide resolved
PR #30235: Size comparison from 1583a24 to bc5f868 Decreases (1 build for efr32)
Full report (72 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
Ping @fengdx4, please note this is failing ZAP now. Please generate if you can! |
PR #30235: Size comparison from cce6c24 to ec23337 Decreases (1 build for efr32)
Full report (72 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
src/app/clusters/microwave-oven-control-server/microwave-oven-control-server.cpp
Outdated
Show resolved
Hide resolved
PR #30235: Size comparison from 0a5f3d3 to c8d83fb Increases (3 builds for esp32, linux)
Decreases (1 build for efr32)
Full report (72 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
…30235) * Microwave oven control cluster implementation * Restyled by whitespace * Restyled by clang-format * updated SDK code of MicrowaveOvenControl for checking all the input values * Restyled by clang-format * fix build err * 1. modified SDK and app layer for MicrowaveOvenControl Cluster to avoid using Attribute accessor; 2. add microwave oven device app * revised device classification ID in examples/microwave-oven-app/linux/include/CHIPProjectAppConfig.h * updated all-cluster-app for returning original * updated zap&matter files for adding new updates * updated the commented code for Microwave Oven Control cluster * Restyled by whitespace * Restyled by clang-format * Restyled by prettier-json * revised copyright and README info * removed 'feature map' in the zcl*.json * Restyled by whitespace * Restyled by clang-format * Restyled by prettier-json * Restyled by prettier-markdown * updated SDK code after reviewed * Restyled by whitespace * Restyled by clang-format * updated zap configuration for BASIC * updated microwave oven control SDK for typo * updated restyled * Restyled by whitespace * Restyled by clang-format * updated Application for microwave oven device * Restyled by whitespace * Restyled by clang-format * revised some reviewed typo * Restyled by clang-format * Restyled by gn * updated App code after reviewed * Restyled by whitespace * Restyled by clang-format * update some comment * Restyled by whitespace * Restyled by gn * recover examples/build_overrides/pigweed_environment.gni * Restyled by gn * fix typos and add unique ptr to stdfunction * update add-more-time command relative functions name * Restyled by whitespace * Restyled by clang-format * regen microwave-oven-app.matter * Restyled by clang-format * update add-more-time command function name * Restyled by clang-format * removed mode code * fix typo in main.cpp * update microwave oven device app and sdk * compiling app * compling sdk * compiling app, sdk * update Microwave Oven Application and SDK code after reviewed * Restyled by whitespace * Restyled by clang-format * update MicrowaveOven App after reviewed * Restyled by whitespace * Restyled by clang-format * updated MicrowaveOvenControl * Restyled by whitespace * fix all_targets_linux_x64.txt * update microwave oven app for program entry * Restyled by whitespace * Restyled by clang-format * move all private after public for microwave-oven-device.h * Restyled by clang-format * update microwave oven app after reviewed * Restyled by whitespace * Restyled by clang-format * update microwave oven mode after reviewed * Restyled by clang-format * update microwave-oven-app.matter * regen ZAP * add api for ModeBase and updated MicrowaveOvenControl * Restyled by whitespace * Restyled by clang-format * update code after reviewed * regen ZAP * Restyled by clang-format --------- Co-authored-by: mideayanghui <[email protected]> Co-authored-by: Restyled.io <[email protected]> Co-authored-by: ex_liangpy4 <[email protected]>
Microwave oven control cluster implementation