-
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
[Chef] Add AirPurifier device in Chef example #31226
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
pullapprove
bot
requested review from
andy31415,
andyg-apple,
anush-apple,
arkq,
bzbarsky-apple,
carol-apple,
cecille,
chrisdecenzo,
chshu,
chulspro,
cliffamzn,
Damian-Nordic,
dhrishi,
harsha-rajendran,
hawk248,
hicklin,
jepenven-silabs,
jmartinez-silabs,
jmeg-sfy,
joonhaengHeo,
jtung-apple,
kkasperczyk-no,
kpschoedel,
ksperling-apple,
lazarkov,
lpbeliveau-silabs,
LuDuda,
mhazley and
mkardous-silabs
January 3, 2024 15:28
1. Simplify the code by a new kLowest vriable 2. Return error when Attribute Read error
erwinpan1
force-pushed
the
add_airpurifier
branch
from
January 12, 2024 16:21
832b479
to
e01f3c0
Compare
andy31415
approved these changes
Jan 12, 2024
PR #31226: Size comparison from 1cb1543 to 93b9788 Decreases (1 build for efr32)
Full report (83 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
erwinpan1
added a commit
to erwinpan1/connectedhomeip
that referenced
this pull request
Feb 6, 2024
* After finish Chef fan control manager * Add Step command to chef airpurifier * Add chef resource monitoring delegates * Support Chef AirPurfier for nRFConnect * Restyled by clang-format * Enable LocalizationConfigurate / TimeFormatLocalization * Remove TimeFormatLocalization and LocalizationConfiguration * Use std::unique_ptr and ValueOr in AirPurifier sample Use std::unique_ptr to manage the pointer in chef-resource-monitoring-delegates.cpp Add ValueOr to chip::Nullable and simplify the code based on it * Restyled by clang-format * Use ValueOr instead of check IsNull() * Fix typo * Replacing with chip::BitMask<ResourceMonitoring::Feature> * Fix typo * Restyled by clang-format * Remove comments * Remove unused comment * Use unique_ptr.reset() instead of unique_ptr = nullptr * Remove unused comment * Not using nullptr when using unique_ptr * Add comments to std::make_unique<ResourceMonitoring::Instance> * Restyled by clang-format * Use chip:BitMask(args...) instead of static_cast * Remove comments cause confusion * Fix speedSetting 1. Simplify the code by a new kLowest vriable 2. Return error when Attribute Read error * Restyled by clang-format * Make mFanControlManager a std::unique_ptr * Rename kLowest to kLowestSpeed * Restyled by clang-format * When decrease, we don't need to check speedMax * Using std::invoke & lambda to return the step * Restyled by clang-format --------- Co-authored-by: Restyled.io <[email protected]>
erwinpan1
added a commit
to erwinpan1/connectedhomeip
that referenced
this pull request
Feb 6, 2024
* After finish Chef fan control manager * Add Step command to chef airpurifier * Add chef resource monitoring delegates * Support Chef AirPurfier for nRFConnect * Restyled by clang-format * Enable LocalizationConfigurate / TimeFormatLocalization * Remove TimeFormatLocalization and LocalizationConfiguration * Use std::unique_ptr and ValueOr in AirPurifier sample Use std::unique_ptr to manage the pointer in chef-resource-monitoring-delegates.cpp Add ValueOr to chip::Nullable and simplify the code based on it * Restyled by clang-format * Use ValueOr instead of check IsNull() * Fix typo * Replacing with chip::BitMask<ResourceMonitoring::Feature> * Fix typo * Restyled by clang-format * Remove comments * Remove unused comment * Use unique_ptr.reset() instead of unique_ptr = nullptr * Remove unused comment * Not using nullptr when using unique_ptr * Add comments to std::make_unique<ResourceMonitoring::Instance> * Restyled by clang-format * Use chip:BitMask(args...) instead of static_cast * Remove comments cause confusion * Fix speedSetting 1. Simplify the code by a new kLowest vriable 2. Return error when Attribute Read error * Restyled by clang-format * Make mFanControlManager a std::unique_ptr * Rename kLowest to kLowestSpeed * Restyled by clang-format * When decrease, we don't need to check speedMax * Using std::invoke & lambda to return the step * Restyled by clang-format --------- Co-authored-by: Restyled.io <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.