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

add kotlin IM read test and enable im test in CI #25636

Merged
merged 2 commits into from
Mar 11, 2023
Merged

add kotlin IM read test and enable im test in CI #25636

merged 2 commits into from
Mar 11, 2023

Conversation

yunhanw-google
Copy link
Contributor

@yunhanw-google yunhanw-google commented Mar 10, 2023 •
edited
Loading

--add kotlin IM read test and enable im test in CI
--use kotlin tlvwriter in kotlin im write/invoke test.

@github-actions
Copy link

github-actions bot commented Mar 10, 2023 •
edited
Loading

PR #25636: Size comparison from 3c6bed3 to b015c6b

Increases (7 builds for bl602, bl702, cc13x2_26x2, cc32xx, nrfconnect, qpg)
platform target config section 3c6bed3 b015c6b change % change
bl602 lighting-app bl602 .text 1029262 1029264 2 0.0
bl602+rpc .text 1059940 1059942 2 0.0
bl702 lighting-app bl702 .debug_info 40868835 40868837 2 0.0
.text 959516 959520 4 0.0
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 681879 681887 8 0.0
.text 592980 592988 8 0.0
cc32xx lock CC3235SF_LAUNCHXL .debug_info 20275631 20275632 1 0.0
nrfconnect all-clusters-minimal-app nrf52840dk_nrf52840 text 773792 773796 4 0.0
qpg lighting-app qpg6105+debug (read/write) 1153452 1153460 8 0.0
.text 600548 600556 8 0.0
Decreases (3 builds for cc13x2_26x2, efr32)
platform target config section 3c6bed3 b015c6b change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read/write) 169376 169368 -8 -0.0
efr32 lighting-app BRD4187C (read/write) 1014652 1014644 -8 -0.0
.text 839024 839016 -8 -0.0
lock-app BRD4161A+wf200 (read/write) 1080876 1080868 -8 -0.0
.text 905016 905008 -8 -0.0
Full report (27 builds for bl602, bl702, cc13x2_26x2, cc32xx, efr32, k32w, linux, mbed, nrfconnect, qpg)
platform target config section 3c6bed3 b015c6b change % change
bl602 lighting-app bl602 (read/write) 1353458 1353458 0 0.0
.bss 94842 94842 0 0.0
.data 9744 9744 0 0.0
.text 1029262 1029264 2 0.0
bl602+rpc (read/write) 1398626 1398626 0 0.0
.bss 102890 102890 0 0.0
.data 10136 10136 0 0.0
.text 1059940 1059942 2 0.0
bl702 lighting-app bl702 (read only) 3358 3358 0 0.0
(read/write) 1192939 1192939 0 0.0
.bleromro 6342 6342 0 0.0
.bleromrw 124 124 0 0.0
.boot2 292 292 0 0.0
.bss 69953 69953 0 0.0
.bss_psram 30160 30160 0 0.0
.comment 48 48 0 0.0
.data 4072 4072 0 0.0
.debug_abbrev 1557348 1557348 0 0.0
.debug_aranges 134656 134656 0 0.0
.debug_frame 493620 493620 0 0.0
.debug_info 40868835 40868837 2 0.0
.debug_line 5297046 5297046 0 0.0
.debug_loc 3428145 3428145 0 0.0
.debug_ranges 373616 373616 0 0.0
.debug_str 3587407 3587407 0 0.0
.hbn 536 536 0 0.0
.hbn_noinit 260 260 0 0.0
.init 342 342 0 0.0
.init_array 144 144 0 0.0
.psram 0 0 0 0.0
.riscv.attributes 47 47 0 0.0
.rodata 108144 108144 0 0.0
.rsvd 2960 2960 0 0.0
.sha_ocram 72 72 0 0.0
.shstrtab 304 304 0 0.0
.stack 2048 2048 0 0.0
.strtab 576906 576906 0 0.0
.symtab 174272 174272 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3358 3358 0 0.0
.text 0 0 0 0.0
959516 959520 4 0.0
bl702+rpc (read only) 3358 3358 0 0.0
(read/write) 1283607 1283607 0 0.0
.bleromro 6342 6342 0 0.0
.bleromrw 124 124 0 0.0
.boot2 292 292 0 0.0
.bss 78001 78001 0 0.0
.bss_psram 30432 30432 0 0.0
.comment 48 48 0 0.0
.data 4616 4616 0 0.0
.debug_abbrev 1709463 1709463 0 0.0
.debug_aranges 142984 142984 0 0.0
.debug_frame 521564 521564 0 0.0
.debug_info 45495697 45495697 0 0.0
.debug_line 5702389 5702389 0 0.0
.debug_loc 3626161 3626161 0 0.0
.debug_ranges 397616 397616 0 0.0
.debug_str 3991603 3991603 0 0.0
.hbn 536 536 0 0.0
.hbn_noinit 260 260 0 0.0
.init 342 342 0 0.0
.init_array 160 160 0 0.0
.psram 0 0 0 0.0
.riscv.attributes 47 47 0 0.0
.rodata 122352 122352 0 0.0
.rsvd 2960 2960 0 0.0
.sha_ocram 72 72 0 0.0
.shstrtab 304 304 0 0.0
.stack 2048 2048 0 0.0
.strtab 638231 638231 0 0.0
.symtab 192768 192768 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3358 3358 0 0.0
.text 0 0 0 0.0
1034860 1034860 0 0.0
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 681879 681887 8 0.0
(read/write) 169376 169368 -8 -0.0
.bss 80948 80948 0 0.0
.data 3352 3352 0 0.0
.rodata 88583 88583 0 0.0
.text 592980 592988 8 0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 645183 645183 0 0.0
(read/write) 157616 157616 0 0.0
.bss 80148 80148 0 0.0
.data 3352 3352 0 0.0
.rodata 78455 78455 0 0.0
.text 566408 566408 0 0.0
lock-ftd LP_CC2652R7 (read only) 679015 679015 0 0.0
(read/write) 169704 169704 0 0.0
.bss 78500 78500 0 0.0
.data 3316 3316 0 0.0
.rodata 76775 76775 0 0.0
.text 601760 601760 0 0.0
lock-mtd LP_CC2652R7 (read only) 664979 664979 0 0.0
(read/write) 178900 178900 0 0.0
.bss 73660 73660 0 0.0
.data 3316 3316 0 0.0
.rodata 103579 103579 0 0.0
.text 560920 560920 0 0.0
pump-app LP_CC2652R7 (read only) 692075 692075 0 0.0
(read/write) 157388 157388 0 0.0
.bss 78476 78476 0 0.0
.data 3280 3280 0 0.0
.rodata 91067 91067 0 0.0
.text 600528 600528 0 0.0
pump-controller-app LP_CC2652R7 (read only) 677011 677011 0 0.0
(read/write) 172588 172588 0 0.0
.bss 78612 78612 0 0.0
.data 3304 3304 0 0.0
.rodata 86939 86939 0 0.0
.text 589592 589592 0 0.0
shell LP_CC2652R7 (read only) 673342 673342 0 0.0
(read/write) 179984 179984 0 0.0
.bss 83020 83020 0 0.0
.data 3348 3348 0 0.0
.rodata 85486 85486 0 0.0
.text 587544 587544 0 0.0
cc32xx lock CC3235SF_LAUNCHXL (read only) 645097 645097 0 0.0
(read/write) 203848 203848 0 0.0
.ARM.attributes 44 44 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 197248 197248 0 0.0
.comment 194 194 0 0.0
.data 1480 1480 0 0.0
.debug_abbrev 930293 930293 0 0.0
.debug_aranges 87384 87384 0 0.0
.debug_frame 300248 300248 0 0.0
.debug_info 20275631 20275632 1 0.0
.debug_line 2660982 2660982 0 0.0
.debug_loc 2804957 2804957 0 0.0
.debug_ranges 283168 283168 0 0.0
.debug_str 3026158 3026158 0 0.0
.ramVecs 780 780 0 0.0
.resetVecs 64 64 0 0.0
.rodata 105993 105993 0 0.0
.shstrtab 232 232 0 0.0
.stab 204 204 0 0.0
.stabstr 441 441 0 0.0
.stack 2048 2048 0 0.0
.strtab 379204 379204 0 0.0
.symtab 256912 256912 0 0.0
.text 0 0 0 0.0
536984 536984 0 0.0
efr32 lighting-app BRD4161A+rs9116 (read/write) 1028492 1028492 0 0.0
.bss 179356 179356 0 0.0
.data 2052 2052 0 0.0
.text 847064 847064 0 0.0
BRD4187C (read/write) 1014652 1014644 -8 -0.0
.bss 148336 148336 0 0.0
.data 2696 2696 0 0.0
.text 839024 839016 -8 -0.0
lock-app BRD4161A+wf200 (read/write) 1080876 1080868 -8 -0.0
.bss 173772 173772 0 0.0
.data 2068 2068 0 0.0
.text 905016 905008 -8 -0.0
window-app BRD4187C (read/write) 1131396 1131396 0 0.0
.bss 132760 132760 0 0.0
.data 2588 2588 0 0.0
.text 971452 971452 0 0.0
k32w contact k32w0+release (read/write) 674176 674176 0 0.0
.bss 65532 65532 0 0.0
.data 2224 2224 0 0.0
.text 575020 575020 0 0.0
light k32w0+release (read/write) 673220 673220 0 0.0
.bss 65196 65196 0 0.0
.data 2212 2212 0 0.0
.text 574412 574412 0 0.0
lock k32w0+release (read/write) 627708 627708 0 0.0
.bss 63268 63268 0 0.0
.data 2136 2136 0 0.0
.text 547288 547288 0 0.0
linux chip-tool-ipv6only arm64 (read only) 12171596 12171596 0 0.0
(read/write) 733432 733432 0 0.0
.bss 34296 34296 0 0.0
.data 3008 3008 0 0.0
.data.rel.ro 675528 675528 0 0.0
.dynamic 560 560 0 0.0
.got 15392 15392 0 0.0
.init 24 24 0 0.0
.init_array 216 216 0 0.0
.rodata 588956 588956 0 0.0
.text 9842052 9842052 0 0.0
thermostat-no-ble arm64 (read only) 2523484 2523484 0 0.0
(read/write) 145240 145240 0 0.0
.bss 56344 56344 0 0.0
.data 1784 1784 0 0.0
.data.rel.ro 77696 77696 0 0.0
.dynamic 560 560 0 0.0
.got 5368 5368 0 0.0
.init 24 24 0 0.0
.init_array 432 432 0 0.0
.rodata 151016 151016 0 0.0
.text 2110080 2110080 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2468104 2468104 0 0.0
.bss 215964 215964 0 0.0
.data 5880 5880 0 0.0
.text 1430748 1430748 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read only) 4 4 0 0.0
(read/write) 1173400 1173400 0 0.0
bss 155557 155557 0 0.0
rodata 132648 132648 0 0.0
text 804428 804428 0 0.0
nrf7002dk_nrf5340_cpuapp (read only) 4 4 0 0.0
(read/write) 1433420 1433420 0 0.0
bss 135297 135297 0 0.0
rodata 228700 228700 0 0.0
text 775348 775348 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read only) 4 4 0 0.0
(read/write) 1118616 1118616 0 0.0
bss 154713 154713 0 0.0
rodata 109464 109464 0 0.0
text 773792 773796 4 0.0
qpg lighting-app qpg6105+debug (read/write) 1153452 1153460 8 0.0
.bss 100076 100076 0 0.0
.data 852 852 0 0.0
.text 600548 600556 8 0.0
lock-app qpg6105+debug (read/write) 1120100 1120100 0 0.0
.bss 96468 96468 0 0.0
.data 864 864 0 0.0
.text 567200 567200 0 0.0

@github-actions
Copy link

PR #25636: Size comparison from 3c6bed3 to 2996592

Increases (2 builds for nrfconnect, qpg)
platform target config section 3c6bed3 2996592 change % change
nrfconnect all-clusters-minimal-app nrf52840dk_nrf52840 text 773792 773796 4 0.0
qpg lighting-app qpg6105+debug (read/write) 1153452 1153460 8 0.0
.text 600548 600556 8 0.0
Full report (7 builds for cc32xx, mbed, nrfconnect, qpg)
platform target config section 3c6bed3 2996592 change % change
cc32xx lock CC3235SF_LAUNCHXL 0 0 0 0.0
(read only) 645097 645097 0 0.0
(read/write) 203848 203848 0 0.0
.ARM.attributes 44 44 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 197248 197248 0 0.0
.comment 194 194 0 0.0
.data 1480 1480 0 0.0
.debug_abbrev 930293 930293 0 0.0
.debug_aranges 87384 87384 0 0.0
.debug_frame 300248 300248 0 0.0
.debug_info 20275631 20275631 0 0.0
.debug_line 2660982 2660982 0 0.0
.debug_loc 2804957 2804957 0 0.0
.debug_ranges 283168 283168 0 0.0
.debug_str 3026158 3026158 0 0.0
.ramVecs 780 780 0 0.0
.resetVecs 64 64 0 0.0
.rodata 105993 105993 0 0.0
.shstrtab 232 232 0 0.0
.stab 204 204 0 0.0
.stabstr 441 441 0 0.0
.stack 2048 2048 0 0.0
.strtab 379204 379204 0 0.0
.symtab 256912 256912 0 0.0
.text 536984 536984 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2468104 2468104 0 0.0
.bss 215964 215964 0 0.0
.data 5880 5880 0 0.0
.text 1430748 1430748 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read only) 4 4 0 0.0
(read/write) 1173400 1173400 0 0.0
bss 155557 155557 0 0.0
rodata 132648 132648 0 0.0
text 804428 804428 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read only) 4 4 0 0.0
(read/write) 1118616 1118616 0 0.0
bss 154713 154713 0 0.0
rodata 109464 109464 0 0.0
text 773792 773796 4 0.0
all-clusters-app nrf7002dk_nrf5340_cpuapp (read only) 4 4 0 0.0
(read/write) 1433420 1433420 0 0.0
bss 135297 135297 0 0.0
rodata 228700 228700 0 0.0
text 775348 775348 0 0.0
qpg lighting-app qpg6105+debug (read/write) 1153452 1153460 8 0.0
.bss 100076 100076 0 0.0
.data 852 852 0 0.0
.text 600548 600556 8 0.0
lock-app qpg6105+debug (read/write) 1120100 1120100 0 0.0
.bss 96468 96468 0 0.0
.data 864 864 0 0.0
.text 567200 567200 0 0.0

yufengwangca
yufengwangca approved these changes Mar 11, 2023
View reviewed changes
.github/workflows/tests.yaml Show resolved Hide resolved
.github/workflows/tests.yaml Show resolved Hide resolved
.github/workflows/tests.yaml Show resolved Hide resolved
.github/workflows/tests.yaml Show resolved Hide resolved
@yunhanw-google yunhanw-google force-pushed the feature/kotlin-im-read-test branch from 2996592 to 15bb77a Compare March 11, 2023 05:07
@github-actions
Copy link

PR #25636: Size comparison from e1381bf to 15bb77a

Increases (1 build for cc32xx)
platform target config section e1381bf 15bb77a change % change
cc32xx lock CC3235SF_LAUNCHXL .debug_info 20275630 20275631 1 0.0
Full report (2 builds for cc32xx, mbed)
platform target config section e1381bf 15bb77a change % change
cc32xx lock CC3235SF_LAUNCHXL 0 0 0 0.0
(read only) 645097 645097 0 0.0
(read/write) 203848 203848 0 0.0
.ARM.attributes 44 44 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 197248 197248 0 0.0
.comment 194 194 0 0.0
.data 1480 1480 0 0.0
.debug_abbrev 930293 930293 0 0.0
.debug_aranges 87384 87384 0 0.0
.debug_frame 300248 300248 0 0.0
.debug_info 20275630 20275631 1 0.0
.debug_line 2660982 2660982 0 0.0
.debug_loc 2804957 2804957 0 0.0
.debug_ranges 283168 283168 0 0.0
.debug_str 3026158 3026158 0 0.0
.ramVecs 780 780 0 0.0
.resetVecs 64 64 0 0.0
.rodata 105993 105993 0 0.0
.shstrtab 232 232 0 0.0
.stab 204 204 0 0.0
.stabstr 441 441 0 0.0
.stack 2048 2048 0 0.0
.strtab 379204 379204 0 0.0
.symtab 256912 256912 0 0.0
.text 536984 536984 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2468104 2468104 0 0.0
.bss 215964 215964 0 0.0
.data 5880 5880 0 0.0
.text 1430748 1430748 0 0.0

@yunhanw-google yunhanw-google force-pushed the feature/kotlin-im-read-test branch from 15bb77a to 5aaea12 Compare March 11, 2023 20:33
@github-actions
Copy link

PR #25636: Size comparison from e1381bf to 5aaea12

Increases (1 build for cc32xx)
platform target config section e1381bf 5aaea12 change % change
cc32xx lock CC3235SF_LAUNCHXL .debug_info 20275630 20275632 2 0.0
Full report (1 build for cc32xx)
platform target config section e1381bf 5aaea12 change % change
cc32xx lock CC3235SF_LAUNCHXL 0 0 0 0.0
(read only) 645097 645097 0 0.0
(read/write) 203848 203848 0 0.0
.ARM.attributes 44 44 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 197248 197248 0 0.0
.comment 194 194 0 0.0
.data 1480 1480 0 0.0
.debug_abbrev 930293 930293 0 0.0
.debug_aranges 87384 87384 0 0.0
.debug_frame 300248 300248 0 0.0
.debug_info 20275630 20275632 2 0.0
.debug_line 2660982 2660982 0 0.0
.debug_loc 2804957 2804957 0 0.0
.debug_ranges 283168 283168 0 0.0
.debug_str 3026158 3026158 0 0.0
.ramVecs 780 780 0 0.0
.resetVecs 64 64 0 0.0
.rodata 105993 105993 0 0.0
.shstrtab 232 232 0 0.0
.stab 204 204 0 0.0
.stabstr 441 441 0 0.0
.stack 2048 2048 0 0.0
.strtab 379204 379204 0 0.0
.symtab 256912 256912 0 0.0
.text 536984 536984 0 0.0

@yunhanw-google yunhanw-google merged commit bc8c9ec into project-chip:master Mar 11, 2023
@yunhanw-google yunhanw-google mentioned this pull request Mar 12, 2023
Merged
lecndav pushed a commit to lecndav/connectedhomeip that referenced this pull request Mar 22, 2023
@yunhanw-google
add kotlin IM read test and enable im test in CI (project-chip#25636)
d3073f9
mwswartwout pushed a commit to mwswartwout/connectedhomeip that referenced this pull request Mar 27, 2023
@yunhanw-google @mwswartwout
add kotlin IM read test and enable im test in CI (project-chip#25636)
a56485d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amitnj amitnj Awaiting requested review from amitnj

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

@arkq arkq Awaiting requested review from arkq

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

@chrisdecenzo chrisdecenzo Awaiting requested review from chrisdecenzo

@chshu chshu Awaiting requested review from chshu

@chulspro chulspro Awaiting requested review from chulspro

@cliffamzn cliffamzn Awaiting requested review from cliffamzn

@CodeChronos928 CodeChronos928 Awaiting requested review from CodeChronos928

@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

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

@gjc13 gjc13 Awaiting requested review from gjc13

@harimau-qirex harimau-qirex Awaiting requested review from harimau-qirex

@harsha-rajendran harsha-rajendran Awaiting requested review from harsha-rajendran

@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

@jmeg-sfy jmeg-sfy Awaiting requested review from jmeg-sfy

@joonhaengHeo joonhaengHeo Awaiting requested review from joonhaengHeo

@jtung-apple jtung-apple Awaiting requested review from jtung-apple

@kcoppock kcoppock Awaiting requested review from kcoppock

@kkasperczyk-no kkasperczyk-no Awaiting requested review from kkasperczyk-no

@ksperling-apple ksperling-apple Awaiting requested review from ksperling-apple

@lazarkov lazarkov Awaiting requested review from lazarkov

@lpbeliveau-silabs lpbeliveau-silabs Awaiting requested review from lpbeliveau-silabs

@LuDuda LuDuda Awaiting requested review from LuDuda

@mrjerryjohns mrjerryjohns Awaiting requested review from mrjerryjohns

@msandstedt msandstedt Awaiting requested review from msandstedt

@nivi-apple nivi-apple Awaiting requested review from nivi-apple

@robszewczyk robszewczyk Awaiting requested review from robszewczyk

@saurabhst saurabhst Awaiting requested review from saurabhst

@selissia selissia Awaiting requested review from selissia

@sharadb-amazon sharadb-amazon Awaiting requested review from sharadb-amazon

@tcarmelveilleux tcarmelveilleux Awaiting requested review from tcarmelveilleux

@tecimovic tecimovic Awaiting requested review from tecimovic

@tehampson tehampson Awaiting requested review from tehampson

@turon turon Awaiting requested review from turon

@vijs vijs Awaiting requested review from vijs

@vivien-apple vivien-apple Awaiting requested review from vivien-apple

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

@xylophone21 xylophone21 Awaiting requested review from xylophone21

@younghak-hwang younghak-hwang Awaiting requested review from younghak-hwang

@bzbarsky-apple bzbarsky-apple Awaiting requested review from bzbarsky-apple

@yufengwangca yufengwangca Awaiting requested review from yufengwangca

Assignees
No one assigned
Labels
examples github review - approved workflows
Projects
[Platform] Android
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@yunhanw-google @yufengwangca @bzbarsky-apple