Oven Chef device type #38108
Merged
Oven Chef device type #38108
Conversation
This file contains hidden or 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
… ignore VID commands
|
|
|
PR #38108: Size comparison from 7d04796 to dc6e072 Full report (5 builds for cc32xx, stm32, tizen)
|
examples/chef/common/clusters/temperature-control/static-supported-temperature-levels.cpp
Outdated
Show resolved
Hide resolved
tcarmelveilleux
requested changes
Mar 26, 2025
Contributor
tcarmelveilleux
left a comment
There was a problem hiding this comment.
- Fix endpoint composition to avoid incrrect children. TC_DeviceBasicComposition test is NOT incorrect, this sample is.
- Use TN feature, not TL, for temperature setting, for the oven. In fact, for the refrigerator as well
This was referenced Mar 28, 2025
19 tasks
|
PR #38108: Size comparison from 779bd5c to e88c53b Full report (14 builds for cc13x4_26x4, cc32xx, nrfconnect, qpg, stm32, tizen)
|
Contributor
Author
|
|
PR #38108: Size comparison from 7a696ce to 899ca44 Full report (3 builds for cc32xx, stm32)
|
|
PR #38108: Size comparison from 0819032 to c98badf Full report (3 builds for cc32xx, stm32)
|
|
PR #38108: Size comparison from 0819032 to 8fe69c1 Full report (75 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
|
PR #38108: Size comparison from e7312a6 to 5830a75 Full report (75 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
...hef/devices/rootnode_oven_temperaturecontrolledcabinet_cooktop_cooksurface_738dd18832.matter
Outdated
Show resolved
Hide resolved
...hef/devices/rootnode_oven_temperaturecontrolledcabinet_cooktop_cooksurface_738dd18832.matter
Outdated
Show resolved
Hide resolved
...hef/devices/rootnode_oven_temperaturecontrolledcabinet_cooktop_cooksurface_738dd18832.matter
Outdated
Show resolved
Hide resolved
|
PR #38108: Size comparison from d170b27 to d3fb9b0 Full report (75 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
|
PR #38108: Size comparison from 2e48cbb to 5ce3214 Full report (3 builds for cc32xx, stm32)
|
|
PR #38108: Size comparison from d2576af to ee1d0c3 Full report (75 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
tcarmelveilleux
approved these changes
Apr 15, 2025
|
PR #38108: Size comparison from bbcf446 to 9b40d99 Full report (75 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
|
PR #38108: Size comparison from 2764129 to 4fa56d9 Full report (75 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
dsavitsky-dsr
pushed a commit
to popovdg/connectedhomeip
that referenced
this pull request
Apr 22, 2025
* Adding ZAP config for Oven Device Type * Adding support for EP3 * Updated ZAP file for Oven device type * Matter file for oven * Added fake entries until issues/37748 is addressed * Updated Oven ZAP file Operational credentials cluster on root node to ignore VID commands * Removed fake entries for SetVidVerificationStatement and SetVidVerificationRequest * Updated temperature control cluster to use TN * Updated temperature control cluster to use TN * Updated temperature control cluster to use TN * Removing Patch for Temerature Level * Updating temperature configurations for Oven
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
Adding ZAP file for Oven Matter Device type
Testing
Ran Conformance test with oven Chef app built for Linux
Command used: python src/python_testing/TC_DeviceConformance.py --qr-code MT:Y.K90AFN00KA0648G00 --dut-node-id 0x20 --commissioning-method on-network
Conformance test Passed: Link to Full Logs: https://docs.google.com/document/d/1u7D80cK-0IwN_0JRZxIn3-9BEynE_Nr_0Kz1Vy_zuhA/edit?tab=t.0
Device Composition test Passed: https://docs.google.com/document/d/1x1HAtldlY2Oi4FSA3z5G6wWSdO3iX5RtEhjAA0YgCLs/edit?tab=t.0
Temporarily Disabled SetVidVerificationStatement and SetVidVerificationRequest from Root Node(0) Operational credentials cluster commands since they are not implemented yet