[pull] master from project-chip:master #597
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
…eDeviceConfigurationChanged (#35906)
* Fixing exception with calling into a dead XPC object * Fixing white space * Restyled by whitespace * Restyled by clang-format --------- Co-authored-by: Restyled.io <commits@restyled.io>
* lighting-app: Remove OccupancySensing Server This change removes the OccupancySensing server from this app as this is against the spec and causes test_TC_IDM_10_1 to fail. With this change, now TC_DeviceBasicComposition.py passes for this example app. * Remove OccupancySensing server from ESP32 example app
* Update darwin-tests.yaml * Update examples-cc13xx_26xx.yaml * Update examples-linux-tv-casting-app.yaml * Update examples-nxp.yaml * Update chef.yaml * Update cirque.yaml * Update darwin.yaml * Update examples-ameba.yaml * Update examples-asr.yaml * Update examples-efr32.yaml * Update examples-esp32.yaml * Update examples-linux-imx.yaml * Update examples-linux-standalone.yaml * Update examples-mw320.yaml * Update examples-nrfconnect.yaml * Update examples-qpg.yaml * Update examples-stm32.yaml * Update examples-tizen.yaml * Update gradle-wrapper-validation.yml * Update spell.yml * Update tests.yaml * Update zap_templates.yaml * Update build.yaml * Update examples-bouffalolab.yaml * Update examples-cc32xx.yaml * Update examples-infineon.yaml * Update examples-linux-arm.yaml * Update examples-telink.yaml * Update full-android.yaml * Update java-tests.yaml * Update lint.yml * Update minimal-build.yaml * Update qemu.yaml * Update unit_integration_test.yaml * Update example-tv-casting-darwin.yaml * Update examples-nuttx.yaml * Update doxygen.yaml * Update build.yaml * Update chef.yaml * Update cirque.yaml * Update darwin-tests.yaml * Update darwin.yaml * Update doxygen.yaml * Update example-tv-casting-darwin.yaml * Update examples-ameba.yaml * Update examples-asr.yaml * Update examples-bouffalolab.yaml * Update examples-cc13xx_26xx.yaml * Update examples-cc32xx.yaml * Update examples-efr32.yaml * Update examples-esp32.yaml * Update examples-infineon.yaml * Update examples-linux-arm.yaml * Update examples-linux-imx.yaml * Update examples-linux-standalone.yaml * Update examples-linux-tv-casting-app.yaml * Update examples-mw320.yaml * Update examples-nrfconnect.yaml * Update examples-nuttx.yaml * Update examples-nxp.yaml * Update examples-qpg.yaml * Update examples-stm32.yaml * Update examples-telink.yaml * Update examples-tizen.yaml * Update full-android.yaml * Update gradle-wrapper-validation.yml * Update java-tests.yaml * Update lint.yml * Update minimal-build.yaml * Update qemu.yaml * Update spell.yml * Update tests.yaml * Update unit_integration_test.yaml * Update zap_templates.yaml * Put back the original efr32 workflow to check runs * Update examples-efr32.yaml --------- Co-authored-by: Andrei Litvin <andy314@gmail.com>
Co-authored-by: cjandhyala <68604034+cjandhyala@users.noreply.github.com>
* [Silabs] Enabling custom board support for matter * Restyled to Make Pretty * Streamline printing to make pretty * Put the assert at the end and add silabs board descriptions * Restyled
* Add TC_CCTRL_2_1 to CI * Add TC_CCTRL_2_2 to CI * Fix copy-paste typo * Allow to override test runner YAML options with command line options * Add TC_CCTRL_2_3 to CI * Run tests on CI * Add MCORE.FS to PICS.yaml
The default for the build environment shoudl be in the out dir since that's already excluded from git. Makes things easier for people following the documentation for the first time who won't necessarily change that directory to their preferred location.
…ute (#35903) * Added a new server-side optional attribute `TCUpdateDeadline` (code 0x09) to the General Commissioning Cluster XML. * This attribute is a 32-bit unsigned integer (`int32u`) and is marked as non-writable. * The attribute is defined as provisional with `apiMaturity="provisional"`. * It includes read access with the privilege set to administer. * Re-generated using ./scripts/tools/zap_regen_all.py
…control (#35862) * Add separate listing for device types * Make unit tests work * Restyle * Fixup include * Fixup include * Make use of DataModel::Provider when doing device type resolution * Add clang-tidy comments * Fix merge error: duplicate declaration * Fix a few more merge conflict markers --------- Co-authored-by: Andrei Litvin <andreilitvin@google.com>
* Add missing feature map bit * Generation
…t ids. (#35920) Makes it easier to tell what commands are being sent, and what attributes are being written.
1) For request paths, clearly mark wildcards instead of logging 0. 2) For all paths except command paths, when we have a value (cluster id, attribute id, event id) that we can find a string representation for, log that string representation. Command paths are a bit complicated because we have to know whether this is a request or a response and we don't have that information in MTRCommandPath at the moment. Will be done in a followup.
IDs are unsigned; we should not be formatting them with %d, because for vendor-prefixed ones that might make them look negative.
Things got a bit too copy/pasted.
#35907) * tv-casting-app fix loading stale binding endpoints from un-needed fabrics * Fixing style issues * Addressed comments by sharadb-amazon * Addressed comments by sharadb-amazon 2 * Addressed comments by sharadb-amazon 3
There isn't actually a _cppCommissioner on the base controller, so that implementation made no sense. In practice, the XPC and concrete controller just have different implementations here, and the base should not try to implement anything. Also fixes the concrete implementation to avoid a sync dispatch by caching the value (which should be immutable) during startup and removes the unnecessary controllerNodeId override from the concrete controller: the base class handles that backwards compat shim already.
…35947) MTROperationalCredentialsDelegate always works with a concrete controller. Make that explicit. At that point, asyncGetCommissionerOnMatterQueue can only get called on an MTRDeviceController from and MTRBaseDevice that was created for an XPC controller. Having that just fail out is perfectly reasonable.
…ithout `ember-compatibility-functions.cpp` (#35919) * Add ability to skip compiling of ember-compatibility-functions * Fix build and make sure that CI will validate a data-model-enabled build * Restyle * Fix compilation * Restyled by clang-format * Typo fix * Fix includes * Rename method to something that seems a bit clearer * Fix pwrpc usage to use datamodel interface * Restyle * Remove todo * Update for attribute is list to support not know value * Update src/app/WriteHandler.h Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> --------- Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>
* [Fabric-Bridge] Replace ScheduleWork with ScheduleLambda * Restyled by whitespace * Restyled by clang-format --------- Co-authored-by: Restyled.io <commits@restyled.io>
…Time ICDs (#35956) * Remove optional spec check * Add static_assert for the slow poll config for SIT ICDs * Add missing include * fix restyler error * fix spacing * Remove client build flag since it is not used
…round (#35950) * Make function argument a reference, to avoid passing in struct data around * Restyled by clang-format --------- Co-authored-by: Restyled.io <commits@restyled.io>
This is only used with concrete controllers from tests, and is already implemented by MTRDeviceController_Concrete. Also removes a test-only class method from MTRDeviceController_Concrete that MTRDeviceController already implements (and has to keep implementing, since the tests use it on MTRDeviceController itself, not an instance).
* Documentation: getting started - changing examples Add a new documentation file that shows how to change, re-gen and recompile examples. * Add to index * Fix spelling
* Fix --int-arg, --bool-arg, etc options parsing * Update CI arguments * Convert to YAML * Support mixed arguments type: multi and append * Restyled by isort --------- Co-authored-by: Restyled.io <commits@restyled.io>
* Use async sleep instead of blocking one * Keep AppServer wrapper class in chip.testing.tasks * Fix typos * Arguments for CI workflow * Require dut_fsa_stdin_pipe options in CI run * Start ICD server on CI run * Read all attributes defined in test plan * Wrap ICD server with IcdAppServerSubprocess * Move app testing wrappers to chip.testing.apps * Add TODO for fixing imports in unit test * Restyled by isort * Remove test from exception list --------- Co-authored-by: Restyled.io <commits@restyled.io>
…enable_leak_checking is true (#35936)
* Add restyled workflow * Update pullapprove to look at the restyle workflow * Update comment * Use same checkout command as the rest of the scripts --------- Co-authored-by: Andrei Litvin <andreilitvin@google.com>
Co-authored-by: Andrei Litvin <andreilitvin@google.com>
* Fix mis-quote in pullapprove * Another fix * Move the check in check_runs --------- Co-authored-by: Andrei Litvin <andreilitvin@google.com>
…anch security (#35970) Co-authored-by: Andrei Litvin <andreilitvin@google.com>
* Add script to start/stop iceccd * Add icecc and ccache * Add ninja-jobs flag * Restyled by whitespace * Restyled by isort * Allow ninja-jobs to be 0 * Fix icecc.sh * Add ninja_jobs to non-GN targets * Add better env var description Co-authored-by: Arkadiusz Bokowy <arkadiusz.bokowy@gmail.com> * Restyled by autopep8 * Fix formatting * Restyled by isort --------- Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: Arkadiusz Bokowy <arkadiusz.bokowy@gmail.com>
…losing the app with Ctrl^C makes it crashes (#35844) * [InteractionModelEngine] Add some checks to Shutdown to prevent crashing if someone calls Shutdown before Init * [CommissioningWindowManager] Ensure Shutdown does nothing if Init has not been called
Co-authored-by: Andrei Litvin <andy314@gmail.com>
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.
See Commits and Changes for more details.
Created by
pull[bot]
Can you help keep this open source service alive? 💖 Please sponsor : )