-
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
[Draft] Re-Implement App Install flow #33982
[Draft] Re-Implement App Install flow #33982
Conversation
PR #33982: Size comparison from 4cdce52 to 112656e Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #33982: Size comparison from ab2907b to 456adc1 Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
@@ -225,6 +225,7 @@ CHIP_ERROR ApplicationBasicAttrAccess::ReadApplicationAttribute(app::AttributeVa | |||
|
|||
CHIP_ERROR ApplicationBasicAttrAccess::ReadStatusAttribute(app::AttributeValueEncoder & aEncoder, Delegate * delegate) | |||
{ | |||
ChipLogProgress(DeviceLayer, "Here 1"); |
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 is not a useful log.
<item name="NotInstalled" value="0x04"/> | ||
<item name="Installing" value="0x05"/> | ||
<item name="InstallationFailed" value="0x06"/> | ||
<item name="Installed" value="0x07"/> |
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 these in the spec. Why are we adding random non-spec bits here?
PR #33982: Size comparison from 9d56786 to 98235b2 Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #33982: Size comparison from 7c04979 to 694132b Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
Re-Implementing application installation flow. Continuation of #33959
Problem
Current application installation flow blocks commissioning and has too crude functionality that cannot provide great UX (providing installation percentage etc.)
Change overview
Tests
Build the TV app using command:
scripts/examples/gn_build_example.sh examples/tv-app/linux out/debug/
Build the TV casting app using command:
scripts/examples/gn_build_example.sh examples/tv-casting-app/linux out/debug/
Start TV App
out/debug/chip-tv-app
Start TV Casting App
out/debug/chip-tv-casting-app
Send Application Launcher command
cast cluster applicationlauncher launch-app 0 1 --Application {"catalogVendorID":123,"applicationID":"com.example.contentapp"}
You should receive pending state:
{ status: 3, data: "" }
Start the app install of content app and send application launcher command again
You should receive installing state:
{ status: 4, data: "" }
In case app install failed and you send application launcher command, you should then receive the state:
{ status: 3, data: "App install failed. Try again" }
---------- TBA --------
Need to add support for tv linux