Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[YAML] Add a mechanism to declare command line variables directly from YAML and support it inside chip-tool #13115

Merged
merged 2 commits into from
Jan 7, 2022
Merged

[YAML] Add a mechanism to declare command line variables directly from YAML and support it inside chip-tool #13115

merged 2 commits into from
Jan 7, 2022

Conversation

vivien-apple
Copy link
Contributor

Problem

YAML tests sometimes needs to use custom variables that can be configured from the command line.

Change overview

  • Add support for declaring variables into the config section of YAML tests

Testing

I have added TestConfigVariables.yaml to chip-tool tests.js. I still need to do the Objective-C bits but there are some other scoped-related issues into the objective C generated content that I needs to solve before.

@vivien-apple vivien-apple self-assigned this Dec 17, 2021
@vivien-apple vivien-apple changed the title Yaml variables [YAML] Add a mechanism to declare command line variables directly from the command line and support it inside chip-tool Dec 17, 2021
@github-actions
Copy link

github-actions bot commented Dec 17, 2021 •
edited
Loading

PR #13115: Size comparison from a9b9fca to adf13ec

Increases above 0.2%:

platform target config section a9b9fca adf13ec change % change
linux chip-tool-ipv6only arm64 .text 5879780 5896196 16416 0.3
Increases (1 build for linux)
platform target config section a9b9fca adf13ec change % change
linux chip-tool-ipv6only arm64 (read only) 6951772 6968668 16896 0.2
(read/write) 324401 324545 144 0.0
.data.rel.ro 208576 208672 96 0.0
.got 56456 56496 40 0.1
.rodata 382580 382740 160 0.0
.text 5879780 5896196 16416 0.3
Full report (19 builds for efr32, k32w, linux, mbed, p6, qpg, telink)
platform target config section a9b9fca adf13ec change % change
efr32 lighting-app BRD4161A (read only) 829904 829904 0 0.0
(read/write) 127352 127352 0 0.0
.bss 125472 125472 0 0.0
.data 1876 1876 0 0.0
.text 829896 829896 0 0.0
BRD4161A+rpc (read only) 817532 817532 0 0.0
(read/write) 144016 144016 0 0.0
.bss 142040 142040 0 0.0
.data 1976 1976 0 0.0
.text 817524 817524 0 0.0
window-app BRD4161A (read only) 803280 803280 0 0.0
(read/write) 126288 126288 0 0.0
.bss 124456 124456 0 0.0
.data 1832 1832 0 0.0
.text 803272 803272 0 0.0
k32w light k32w061+release (read/write) 648664 648664 0 0.0
.bss 76480 76480 0 0.0
.data 1904 1904 0 0.0
.text 564480 564480 0 0.0
lock k32w061+release (read/write) 633380 633380 0 0.0
.bss 76200 76200 0 0.0
.data 1860 1860 0 0.0
.text 549520 549520 0 0.0
linux chip-tool-ipv6only arm64 (read only) 6951772 6968668 16896 0.2
(read/write) 324401 324545 144 0.0
.bss 54577 54577 0 0.0
.data 1096 1096 0 0.0
.data.rel.ro 208576 208672 96 0.0
.dynamic 560 560 0 0.0
.got 56456 56496 40 0.1
.init 24 24 0 0.0
.init_array 168 168 0 0.0
.rodata 382580 382740 160 0.0
.text 5879780 5896196 16416 0.3
thermostat-no-ble arm64 (read only) 1996756 1996756 0 0.0
(read/write) 143953 143953 0 0.0
.bss 64321 64321 0 0.0
.data 880 880 0 0.0
.data.rel.ro 72024 72024 0 0.0
.dynamic 560 560 0 0.0
.got 3840 3840 0 0.0
.init 24 24 0 0.0
.init_array 288 288 0 0.0
.rodata 128196 128196 0 0.0
.text 1657072 1657072 0 0.0
mbed all-clusters-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2334424 2334424 0 0.0
.bss 189068 189068 0 0.0
.data 5264 5264 0 0.0
.text 1297000 1297000 0 0.0
lighting-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2329000 2329000 0 0.0
.bss 180896 180896 0 0.0
.data 5544 5544 0 0.0
.text 1291600 1291600 0 0.0
lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2302032 2302032 0 0.0
.bss 179944 179944 0 0.0
.data 5536 5536 0 0.0
.text 1264632 1264632 0 0.0
pigweed-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 1140008 1140008 0 0.0
.bss 11756 11756 0 0.0
.data 4368 4368 0 0.0
.text 103392 103392 0 0.0
shell CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2053688 2053688 0 0.0
.bss 156972 156972 0 0.0
.data 4864 4864 0 0.0
.text 1016288 1016288 0 0.0
p6 all-clusters-app default (read/write) 2385072 2385072 0 0.0
.bss 117260 117260 0 0.0
.data 2544 2544 0 0.0
.text 1343336 1343336 0 0.0
light-app default (read/write) 2324456 2324456 0 0.0
.bss 106152 106152 0 0.0
.data 2384 2384 0 0.0
.text 1282720 1282720 0 0.0
lock-app default (read/write) 2296648 2296648 0 0.0
.bss 105032 105032 0 0.0
.data 2336 2336 0 0.0
.text 1254912 1254912 0 0.0
qpg lighting-app qpg6105+debug (read only) 532172 532172 0 0.0
(read/write) 146936 146936 0 0.0
.bss 86816 86816 0 0.0
.data 1004 1004 0 0.0
.text 526852 526852 0 0.0
lock-app qpg6105+debug (read only) 503836 503836 0 0.0
(read/write) 146940 146940 0 0.0
.bss 85952 85952 0 0.0
.data 952 952 0 0.0
.text 498516 498516 0 0.0
persistent-storage-app qpg6105+debug (read only) 106448 106448 0 0.0
(read/write) 146938 146938 0 0.0
.bss 36146 36146 0 0.0
.data 288 288 0 0.0
.text 101128 101128 0 0.0
telink lighting-app tlsr9518adk80d (read/write) 830638 830638 0 0.0
bss 87040 87040 0 0.0
noinit 37160 37160 0 0.0
text 578832 578832 0 0.0

@github-actions
Copy link

github-actions bot commented Dec 17, 2021 •
edited
Loading

PR #13115: Size comparison from a9b9fca to dbe68e4

Increases above 0.2%:

platform target config section a9b9fca dbe68e4 change % change
linux chip-tool-ipv6only arm64 .text 5879780 5896196 16416 0.3
Increases (1 build for linux)
platform target config section a9b9fca dbe68e4 change % change
linux chip-tool-ipv6only arm64 (read only) 6951772 6968668 16896 0.2
(read/write) 324401 324545 144 0.0
.data.rel.ro 208576 208672 96 0.0
.got 56456 56496 40 0.1
.rodata 382580 382740 160 0.0
.text 5879780 5896196 16416 0.3
Full report (31 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
platform target config section a9b9fca dbe68e4 change % change
efr32 lighting-app BRD4161A (read only) 829904 829904 0 0.0
(read/write) 127352 127352 0 0.0
.bss 125472 125472 0 0.0
.data 1876 1876 0 0.0
.text 829896 829896 0 0.0
BRD4161A+rpc (read only) 817532 817532 0 0.0
(read/write) 144016 144016 0 0.0
.bss 142040 142040 0 0.0
.data 1976 1976 0 0.0
.text 817524 817524 0 0.0
window-app BRD4161A (read only) 803280 803280 0 0.0
(read/write) 126288 126288 0 0.0
.bss 124456 124456 0 0.0
.data 1832 1832 0 0.0
.text 803272 803272 0 0.0
esp32 all-clusters-app c3devkit (read only) 877496 877496 0 0.0
(read/write) 1313042 1313042 0 0.0
.dram0.bss 69784 69784 0 0.0
.dram0.data 14220 14220 0 0.0
.flash.rodata 175976 175976 0 0.0
.flash.text 877496 877496 0 0.0
.iram0.text 62254 62254 0 0.0
m5stack (read only) 938355 938355 0 0.0
(read/write) 442144 442144 0 0.0
.dram0.bss 74280 74280 0 0.0
.dram0.data 34056 34056 0 0.0
.flash.rodata 202800 202800 0 0.0
.flash.text 932971 932971 0 0.0
.iram0.text 122671 122671 0 0.0
k32w light k32w061+release (read/write) 648664 648664 0 0.0
.bss 76480 76480 0 0.0
.data 1904 1904 0 0.0
.text 564480 564480 0 0.0
lock k32w061+release (read/write) 633380 633380 0 0.0
.bss 76200 76200 0 0.0
.data 1860 1860 0 0.0
.text 549520 549520 0 0.0
linux chip-tool-ipv6only arm64 (read only) 6951772 6968668 16896 0.2
(read/write) 324401 324545 144 0.0
.bss 54577 54577 0 0.0
.data 1096 1096 0 0.0
.data.rel.ro 208576 208672 96 0.0
.dynamic 560 560 0 0.0
.got 56456 56496 40 0.1
.init 24 24 0 0.0
.init_array 168 168 0 0.0
.rodata 382580 382740 160 0.0
.text 5879780 5896196 16416 0.3
thermostat-no-ble arm64 (read only) 1996756 1996756 0 0.0
(read/write) 143953 143953 0 0.0
.bss 64321 64321 0 0.0
.data 880 880 0 0.0
.data.rel.ro 72024 72024 0 0.0
.dynamic 560 560 0 0.0
.got 3840 3840 0 0.0
.init 24 24 0 0.0
.init_array 288 288 0 0.0
.rodata 128196 128196 0 0.0
.text 1657072 1657072 0 0.0
mbed all-clusters-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2334424 2334424 0 0.0
.bss 189068 189068 0 0.0
.data 5264 5264 0 0.0
.text 1297000 1297000 0 0.0
lighting-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2329000 2329000 0 0.0
.bss 180896 180896 0 0.0
.data 5544 5544 0 0.0
.text 1291600 1291600 0 0.0
lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2302032 2302032 0 0.0
.bss 179944 179944 0 0.0
.data 5536 5536 0 0.0
.text 1264632 1264632 0 0.0
pigweed-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 1140008 1140008 0 0.0
.bss 11756 11756 0 0.0
.data 4368 4368 0 0.0
.text 103392 103392 0 0.0
shell CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2053688 2053688 0 0.0
.bss 156972 156972 0 0.0
.data 4864 4864 0 0.0
.text 1016288 1016288 0 0.0
nrfconnect lighting-app nrf52840dk_nrf52840 (read/write) 935915 935915 0 0.0
bss 118400 118400 0 0.0
rodata 108120 108120 0 0.0
text 631844 631844 0 0.0
nrf52840dk_nrf52840+rpc (read/write) 922315 922315 0 0.0
bss 115444 115444 0 0.0
rodata 101536 101536 0 0.0
text 627156 627156 0 0.0
nrf5340dk_nrf5340_cpuapp (read/write) 859678 859678 0 0.0
bss 116684 116684 0 0.0
rodata 103044 103044 0 0.0
text 559284 559284 0 0.0
lock-app nrf52840dk_nrf52840 (read/write) 908059 908059 0 0.0
bss 117588 117588 0 0.0
rodata 103424 103424 0 0.0
text 609668 609668 0 0.0
nrf5340dk_nrf5340_cpuapp (read/write) 831974 831974 0 0.0
bss 115900 115900 0 0.0
rodata 98388 98388 0 0.0
text 537152 537152 0 0.0
pigweed-app nrf52840dk_nrf52840 (read/write) 542351 542351 0 0.0
bss 52588 52588 0 0.0
rodata 50668 50668 0 0.0
text 376892 376892 0 0.0
pump-app nrf52840dk_nrf52840 (read/write) 909355 909355 0 0.0
bss 117348 117348 0 0.0
rodata 103640 103640 0 0.0
text 610908 610908 0 0.0
pump-controller-app nrf52840dk_nrf52840 (read/write) 906139 906139 0 0.0
bss 117376 117376 0 0.0
rodata 102896 102896 0 0.0
text 608416 608416 0 0.0
shell nrf52840dk_nrf52840 (read/write) 796079 796079 0 0.0
bss 109464 109464 0 0.0
rodata 78096 78096 0 0.0
text 532048 532048 0 0.0
nrf5340dk_nrf5340_cpuapp (read/write) 708710 708710 0 0.0
bss 107352 107352 0 0.0
rodata 72396 72396 0 0.0
text 449544 449544 0 0.0
p6 all-clusters-app default (read/write) 2385072 2385072 0 0.0
.bss 117260 117260 0 0.0
.data 2544 2544 0 0.0
.text 1343336 1343336 0 0.0
light-app default (read/write) 2324456 2324456 0 0.0
.bss 106152 106152 0 0.0
.data 2384 2384 0 0.0
.text 1282720 1282720 0 0.0
lock-app default (read/write) 2296648 2296648 0 0.0
.bss 105032 105032 0 0.0
.data 2336 2336 0 0.0
.text 1254912 1254912 0 0.0
qpg lighting-app qpg6105+debug (read only) 532172 532172 0 0.0
(read/write) 146936 146936 0 0.0
.bss 86816 86816 0 0.0
.data 1004 1004 0 0.0
.text 526852 526852 0 0.0
lock-app qpg6105+debug (read only) 503836 503836 0 0.0
(read/write) 146940 146940 0 0.0
.bss 85952 85952 0 0.0
.data 952 952 0 0.0
.text 498516 498516 0 0.0
persistent-storage-app qpg6105+debug (read only) 106448 106448 0 0.0
(read/write) 146938 146938 0 0.0
.bss 36146 36146 0 0.0
.data 288 288 0 0.0
.text 101128 101128 0 0.0
telink lighting-app tlsr9518adk80d (read/write) 830638 830638 0 0.0
bss 87040 87040 0 0.0
noinit 37160 37160 0 0.0
text 578832 578832 0 0.0

@stale
Copy link

stale bot commented Dec 25, 2021

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the stale Stale issue or PR label Dec 25, 2021
@stale
Copy link

stale bot commented Jan 1, 2022

This stale pull request has been automatically closed. Thank you for your contributions.

@stale stale bot closed this Jan 1, 2022
@vivien-apple vivien-apple reopened this Jan 3, 2022
@stale stale bot removed the stale Stale issue or PR label Jan 3, 2022
@vivien-apple vivien-apple changed the title [YAML] Add a mechanism to declare command line variables directly from the command line and support it inside chip-tool [YAML] Add a mechanism to declare command line variables directly from YAML and support it inside chip-tool Jan 4, 2022
@github-actions
Copy link

github-actions bot commented Jan 4, 2022 •
edited
Loading

PR #13115: Size comparison from b0af6ba to 29b29d4

Increases above 0.2%:

platform target config section b0af6ba 29b29d4 change % change
linux chip-tool-ipv6only arm64 (read only) 7014204 7031212 17008 0.2
.text 5938436 5954964 16528 0.3
Increases (1 build for linux)
platform target config section b0af6ba 29b29d4 change % change
linux chip-tool-ipv6only arm64 (read only) 7014204 7031212 17008 0.2
(read/write) 324865 325009 144 0.0
.data.rel.ro 208984 209080 96 0.0
.got 56880 56920 40 0.1
.rodata 383332 383492 160 0.0
.text 5938436 5954964 16528 0.3
Full report (31 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
platform target config section b0af6ba 29b29d4 change % change
efr32 lighting-app BRD4161A (read only) 828924 828924 0 0.0
(read/write) 127052 127052 0 0.0
.bss 125176 125176 0 0.0
.data 1876 1876 0 0.0
.text 828916 828916 0 0.0
BRD4161A+rpc (read only) 816552 816552 0 0.0
(read/write) 143720 143720 0 0.0
.bss 141744 141744 0 0.0
.data 1976 1976 0 0.0
.text 816544 816544 0 0.0
window-app BRD4161A (read only) 802372 802372 0 0.0
(read/write) 125992 125992 0 0.0
.bss 124160 124160 0 0.0
.data 1832 1832 0 0.0
.text 802364 802364 0 0.0
esp32 all-clusters-app c3devkit (read only) 888138 888138 0 0.0
(read/write) 1313666 1313666 0 0.0
.dram0.bss 69264 69264 0 0.0
.dram0.data 14228 14228 0 0.0
.flash.rodata 177112 177112 0 0.0
.flash.text 888138 888138 0 0.0
.iram0.text 62254 62254 0 0.0
m5stack (read only) 948215 948215 0 0.0
(read/write) 445252 445252 0 0.0
.dram0.bss 73760 73760 0 0.0
.dram0.data 34056 34056 0 0.0
.flash.rodata 206428 206428 0 0.0
.flash.text 942831 942831 0 0.0
.iram0.text 122671 122671 0 0.0
k32w light k32w061+release (read/write) 647440 647440 0 0.0
.bss 76184 76184 0 0.0
.data 1904 1904 0 0.0
.text 563552 563552 0 0.0
lock k32w061+release (read/write) 632264 632264 0 0.0
.bss 75896 75896 0 0.0
.data 1860 1860 0 0.0
.text 548708 548708 0 0.0
linux chip-tool-ipv6only arm64 (read only) 7014204 7031212 17008 0.2
(read/write) 324865 325009 144 0.0
.bss 54209 54209 0 0.0
.data 1096 1096 0 0.0
.data.rel.ro 208984 209080 96 0.0
.dynamic 560 560 0 0.0
.got 56880 56920 40 0.1
.init 24 24 0 0.0
.init_array 168 168 0 0.0
.rodata 383332 383492 160 0.0
.text 5938436 5954964 16528 0.3
thermostat-no-ble arm64 (read only) 2021852 2021852 0 0.0
(read/write) 143937 143937 0 0.0
.bss 63985 63985 0 0.0
.data 880 880 0 0.0
.data.rel.ro 72200 72200 0 0.0
.dynamic 560 560 0 0.0
.got 3936 3936 0 0.0
.init 24 24 0 0.0
.init_array 296 296 0 0.0
.rodata 128476 128476 0 0.0
.text 1680848 1680848 0 0.0
mbed all-clusters-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2345456 2345456 0 0.0
.bss 188524 188524 0 0.0
.data 5304 5304 0 0.0
.text 1308032 1308032 0 0.0
lighting-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2329040 2329040 0 0.0
.bss 180608 180608 0 0.0
.data 5552 5552 0 0.0
.text 1291640 1291640 0 0.0
lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2302200 2302200 0 0.0
.bss 179656 179656 0 0.0
.data 5544 5544 0 0.0
.text 1264800 1264800 0 0.0
pigweed-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 1140008 1140008 0 0.0
.bss 11756 11756 0 0.0
.data 4368 4368 0 0.0
.text 103392 103392 0 0.0
shell CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2053888 2053888 0 0.0
.bss 156972 156972 0 0.0
.data 4864 4864 0 0.0
.text 1016488 1016488 0 0.0
nrfconnect lighting-app nrf52840dk_nrf52840 (read/write) 935247 935247 0 0.0
bss 118172 118172 0 0.0
rodata 107996 107996 0 0.0
text 631528 631528 0 0.0
nrf52840dk_nrf52840+rpc (read/write) 921651 921651 0 0.0
bss 115216 115216 0 0.0
rodata 101416 101416 0 0.0
text 626840 626840 0 0.0
nrf5340dk_nrf5340_cpuapp (read/write) 859014 859014 0 0.0
bss 116456 116456 0 0.0
rodata 102924 102924 0 0.0
text 558964 558964 0 0.0
lock-app nrf52840dk_nrf52840 (read/write) 907507 907507 0 0.0
bss 117360 117360 0 0.0
rodata 103304 103304 0 0.0
text 609460 609460 0 0.0
nrf5340dk_nrf5340_cpuapp (read/write) 831422 831422 0 0.0
bss 115672 115672 0 0.0
rodata 98268 98268 0 0.0
text 536940 536940 0 0.0
pigweed-app nrf52840dk_nrf52840 (read/write) 542351 542351 0 0.0
bss 52588 52588 0 0.0
rodata 50668 50668 0 0.0
text 376892 376892 0 0.0
pump-app nrf52840dk_nrf52840 (read/write) 908739 908739 0 0.0
bss 117120 117120 0 0.0
rodata 103520 103520 0 0.0
text 610676 610676 0 0.0
pump-controller-app nrf52840dk_nrf52840 (read/write) 905587 905587 0 0.0
bss 117148 117148 0 0.0
rodata 102776 102776 0 0.0
text 608208 608208 0 0.0
shell nrf52840dk_nrf52840 (read/write) 796871 796871 0 0.0
bss 109536 109536 0 0.0
rodata 78200 78200 0 0.0
text 532680 532680 0 0.0
nrf5340dk_nrf5340_cpuapp (read/write) 709502 709502 0 0.0
bss 107424 107424 0 0.0
rodata 72500 72500 0 0.0
text 450172 450172 0 0.0
p6 all-clusters-app default (read/write) 2399776 2399776 0 0.0
.bss 116732 116732 0 0.0
.data 2584 2584 0 0.0
.text 1358040 1358040 0 0.0
light-app default (read/write) 2323680 2323680 0 0.0
.bss 105856 105856 0 0.0
.data 2384 2384 0 0.0
.text 1281944 1281944 0 0.0
lock-app default (read/write) 2295880 2295880 0 0.0
.bss 104736 104736 0 0.0
.data 2336 2336 0 0.0
.text 1254144 1254144 0 0.0
qpg lighting-app qpg6105+debug (read only) 531096 531096 0 0.0
(read/write) 146936 146936 0 0.0
.bss 86656 86656 0 0.0
.data 1004 1004 0 0.0
.text 525776 525776 0 0.0
lock-app qpg6105+debug (read only) 502876 502876 0 0.0
(read/write) 146940 146940 0 0.0
.bss 85792 85792 0 0.0
.data 952 952 0 0.0
.text 497556 497556 0 0.0
persistent-storage-app qpg6105+debug (read only) 106448 106448 0 0.0
(read/write) 146938 146938 0 0.0
.bss 36146 36146 0 0.0
.data 288 288 0 0.0
.text 101128 101128 0 0.0
telink lighting-app tlsr9518adk80d (read/write) 831754 831754 0 0.0
bss 86840 86840 0 0.0
noinit 37160 37160 0 0.0
text 580232 580232 0 0.0

@vivien-apple
[YAML] Add a mechanism to declare command line variables directly fro…
78b3402 
…m the command line and support it inside chip-tool
@github-actions
Copy link

github-actions bot commented Jan 7, 2022 •
edited
Loading

PR #13115: Size comparison from 36a759d to c822ee0

Full report (6 builds for k32w, qpg, telink)
platform target config section 36a759d c822ee0 change % change
k32w light k32w061+release (read/write) 655772 655772 0 0.0
.bss 76864 76864 0 0.0
.data 1848 1848 0 0.0
.text 571260 571260 0 0.0
lock k32w061+release (read/write) 660032 660032 0 0.0
.bss 77160 77160 0 0.0
.data 1868 1868 0 0.0
.text 575204 575204 0 0.0
qpg lighting-app qpg6105+debug (read only) 533268 533268 0 0.0
(read/write) 146936 146936 0 0.0
.bss 86688 86688 0 0.0
.data 1004 1004 0 0.0
.text 527948 527948 0 0.0
lock-app qpg6105+debug (read only) 505044 505044 0 0.0
(read/write) 146940 146940 0 0.0
.bss 85824 85824 0 0.0
.data 952 952 0 0.0
.text 499724 499724 0 0.0
persistent-storage-app qpg6105+debug (read only) 106448 106448 0 0.0
(read/write) 146938 146938 0 0.0
.bss 36146 36146 0 0.0
.data 288 288 0 0.0
.text 101128 101128 0 0.0
telink lighting-app tlsr9518adk80d (read/write) 834102 834102 0 0.0
bss 86864 86864 0 0.0
noinit 37160 37160 0 0.0
text 582240 582240 0 0.0

@github-actions
Copy link

github-actions bot commented Jan 7, 2022 •
edited
Loading

PR #13115: Size comparison from 36a759d to 13c349c

Increases above 0.2%:

platform target config section 36a759d 13c349c change % change
linux chip-tool-ipv6only arm64 .text 5956932 5973332 16400 0.3
Increases (1 build for linux)
platform target config section 36a759d 13c349c change % change
linux chip-tool-ipv6only arm64 (read only) 7033996 7050876 16880 0.2
(read/write) 325073 325217 144 0.0
.data.rel.ro 209064 209160 96 0.0
.got 56992 57032 40 0.1
.rodata 384292 384452 160 0.0
.text 5956932 5973332 16400 0.3
Full report (16 builds for k32w, linux, mbed, p6, qpg, telink)
platform target config section 36a759d 13c349c change % change
k32w light k32w061+release (read/write) 655772 655772 0 0.0
.bss 76864 76864 0 0.0
.data 1848 1848 0 0.0
.text 571260 571260 0 0.0
lock k32w061+release (read/write) 660032 660032 0 0.0
.bss 77160 77160 0 0.0
.data 1868 1868 0 0.0
.text 575204 575204 0 0.0
linux chip-tool-ipv6only arm64 (read only) 7033996 7050876 16880 0.2
(read/write) 325073 325217 144 0.0
.bss 54241 54241 0 0.0
.data 1096 1096 0 0.0
.data.rel.ro 209064 209160 96 0.0
.dynamic 560 560 0 0.0
.got 56992 57032 40 0.1
.init 24 24 0 0.0
.init_array 168 168 0 0.0
.rodata 384292 384452 160 0.0
.text 5956932 5973332 16400 0.3
thermostat-no-ble arm64 (read only) 2031036 2031036 0 0.0
(read/write) 144193 144193 0 0.0
.bss 64033 64033 0 0.0
.data 880 880 0 0.0
.data.rel.ro 72392 72392 0 0.0
.dynamic 560 560 0 0.0
.got 3952 3952 0 0.0
.init 24 24 0 0.0
.init_array 296 296 0 0.0
.rodata 128844 128844 0 0.0
.text 1689280 1689280 0 0.0
mbed all-clusters-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2349024 2349024 0 0.0
.bss 188812 188812 0 0.0
.data 5312 5312 0 0.0
.text 1311600 1311600 0 0.0
lighting-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2330912 2330912 0 0.0
.bss 180632 180632 0 0.0
.data 5552 5552 0 0.0
.text 1293512 1293512 0 0.0
lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2304072 2304072 0 0.0
.bss 179680 179680 0 0.0
.data 5544 5544 0 0.0
.text 1266672 1266672 0 0.0
pigweed-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 1139712 1139712 0 0.0
.bss 11756 11756 0 0.0
.data 4368 4368 0 0.0
.text 103096 103096 0 0.0
shell CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2053888 2053888 0 0.0
.bss 156972 156972 0 0.0
.data 4864 4864 0 0.0
.text 1016488 1016488 0 0.0
p6 all-clusters-app default (read/write) 2404872 2404872 0 0.0
.bss 117020 117020 0 0.0
.data 2592 2592 0 0.0
.text 1363136 1363136 0 0.0
light-app default (read/write) 2326832 2326832 0 0.0
.bss 105888 105888 0 0.0
.data 2384 2384 0 0.0
.text 1285096 1285096 0 0.0
lock-app default (read/write) 2299032 2299032 0 0.0
.bss 104768 104768 0 0.0
.data 2336 2336 0 0.0
.text 1257296 1257296 0 0.0
qpg lighting-app qpg6105+debug (read only) 533268 533268 0 0.0
(read/write) 146936 146936 0 0.0
.bss 86688 86688 0 0.0
.data 1004 1004 0 0.0
.text 527948 527948 0 0.0
lock-app qpg6105+debug (read only) 505044 505044 0 0.0
(read/write) 146940 146940 0 0.0
.bss 85824 85824 0 0.0
.data 952 952 0 0.0
.text 499724 499724 0 0.0
persistent-storage-app qpg6105+debug (read only) 106448 106448 0 0.0
(read/write) 146938 146938 0 0.0
.bss 36146 36146 0 0.0
.data 288 288 0 0.0
.text 101128 101128 0 0.0
telink lighting-app tlsr9518adk80d (read/write) 834102 834102 0 0.0
bss 86864 86864 0 0.0
noinit 37160 37160 0 0.0
text 582240 582240 0 0.0

@bzbarsky-apple
Copy link
Contributor

Fast-tracking test-only change that's been up for a while.

@bzbarsky-apple bzbarsky-apple merged commit 41758c0 into project-chip:master Jan 7, 2022
step0035 pushed a commit to hank820/connectedhomeip that referenced this pull request Feb 8, 2022
@vivien-apple @step0035
[YAML] Add a mechanism to declare command line variables directly fro…
4304101 
…m YAML and support it inside chip-tool (project-chip#13115)

* [YAML] Add a mechanism to declare command line variables directly from the command line and support it inside chip-tool

* Update generated tests content
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jmeg-sfy jmeg-sfy jmeg-sfy approved these changes

@bzbarsky-apple bzbarsky-apple bzbarsky-apple approved these changes

@andy31415 andy31415 Awaiting requested review from andy31415

@anush-apple anush-apple Awaiting requested review from anush-apple

@austinh0 austinh0 Awaiting requested review from austinh0

@balducci-apple balducci-apple Awaiting requested review from balducci-apple

@Byungjoo-Lee Byungjoo-Lee Awaiting requested review from Byungjoo-Lee

@carol-apple carol-apple Awaiting requested review from carol-apple

@cecille cecille Awaiting requested review from cecille

@chrisdecenzo chrisdecenzo Awaiting requested review from chrisdecenzo

@chshu chshu Awaiting requested review from chshu

@chulspro chulspro Awaiting requested review from chulspro

@Damian-Nordic Damian-Nordic Awaiting requested review from Damian-Nordic

@dhrishi dhrishi Awaiting requested review from dhrishi

@electrocucaracha electrocucaracha Awaiting requested review from electrocucaracha

@emargolis emargolis Awaiting requested review from emargolis

@erjiaqing erjiaqing Awaiting requested review from erjiaqing

@franck-apple franck-apple Awaiting requested review from franck-apple

@gjc13 gjc13 Awaiting requested review from gjc13

@hawk248 hawk248 Awaiting requested review from hawk248

@jelderton jelderton Awaiting requested review from jelderton

@jepenven-silabs jepenven-silabs Awaiting requested review from jepenven-silabs

@jmartinez-silabs jmartinez-silabs Awaiting requested review from jmartinez-silabs

@kpschoedel kpschoedel Awaiting requested review from kpschoedel

@LuDuda LuDuda Awaiting requested review from LuDuda

@lzgrablic02 lzgrablic02 Awaiting requested review from lzgrablic02

@mlepage-google mlepage-google Awaiting requested review from mlepage-google

@msandstedt msandstedt Awaiting requested review from msandstedt

@mspang mspang Awaiting requested review from mspang

@pan-apple pan-apple Awaiting requested review from pan-apple

@robszewczyk robszewczyk Awaiting requested review from robszewczyk

@sagar-apple sagar-apple Awaiting requested review from sagar-apple

@saurabhst saurabhst Awaiting requested review from saurabhst

@selissia selissia Awaiting requested review from selissia

@tecimovic tecimovic Awaiting requested review from tecimovic

@vijs vijs Awaiting requested review from vijs

@wbschiller wbschiller Awaiting requested review from wbschiller

@woody-apple woody-apple Awaiting requested review from woody-apple

@xylophone21 xylophone21 Awaiting requested review from xylophone21

Assignees

@vivien-apple vivien-apple

Labels
app examples fast track review - pending
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@vivien-apple @bzbarsky-apple @jmeg-sfy