Add python building build capabilities to build_examples.py #17341

andy31415 · 2022-04-13T16:43:06Z

Problem

Looking to have a very uniform build logic by using build_examples.

Change overview

Adds python binding compile support - both host and cross compile if applicable.

Testing

CI and unit tests were updated. Compiled locally too.

github-actions · 2022-04-13T17:00:38Z

PR #17341: Size comparison from 94e8197 to a87f565

Full report (22 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)

platform	target	config	section	`94e8197`	`a87f565`
cc13x2_26x2	lock-ftd	LP_CC2652R7	(read only)	640435	640435
			(read/write)	151212	151212
			.bss	74144	74144
			.data	3212	3212
			.rodata	80227	80227
			.text	559716	559716
	lock-mtd	LP_CC2652R7	(read only)	589171	589171
			(read/write)	146932	146932
			.bss	69864	69864
			.data	3212	3212
			.rodata	80107	80107
			.text	508572	508572
	pump-app	LP_CC2652R7	(read only)	648527	648527
			(read/write)	152508	152508
			.bss	74640	74640
			.data	3244	3244
			.rodata	75415	75415
			.text	572624	572624
	pump-controller-app	LP_CC2652R7	(read only)	642499	642499
			(read/write)	152176	152176
			.bss	74344	74344
			.data	3208	3208
			.rodata	79051	79051
			.text	562960	562960
cyw30739	light	cyw930739m2evb_01	(read/write)	619042	619042
			.app_xip_area	525752	525752
			.bss	75956	75956
			.data	684	684
			.rodata	0	0
			.text	0	0
	lock	cyw930739m2evb_01	(read/write)	576646	576646
			.app_xip_area	484892	484892
			.bss	74452	74452
			.data	648	648
			.rodata	0	0
			.text	0	0
	ota-requestor-no-progress-logging	cyw930739m2evb_01	(read/write)	565170	565170
			.app_xip_area	463788	463788
			.bss	83784	83784
			.data	564	564
			.rodata	0	0
			.text	112	112
efr32	lighting-app	BRD4161A	(read only)	908052	908052
			(read/write)	133144	133144
			.bss	131104	131104
			.data	2040	2040
			.text	908044	908044
		BRD4161A+rpc	(read only)	942420	942420
			(read/write)	149828	149828
			.bss	147584	147584
			.data	2244	2244
			.text	942412	942412
	window-app	BRD4161A	(read only)	844708	844708
			(read/write)	131148	131148
			.bss	129200	129200
			.data	1948	1948
			.text	844700	844700
esp32	all-clusters-app	c3devkit	(read only)	979972	979972
			(read/write)	`1397578`	`1397578`
			.dram0.bss	62624	62624
			.dram0.data	14420	14420
			.flash.rodata	201616	201616
			.flash.text	979972	979972
			.iram0.text	62016	62016
		m5stack	(read only)	1035459	1035459
			(read/write)	465316	465316
			.dram0.bss	68144	68144
			.dram0.data	34152	34152
			.flash.rodata	231184	231184
			.flash.text	1030075	1030075
			.iram0.text	123107	123107
k32w	light	k32w061+release	(read/write)	687260	687260
			.bss	78128	78128
			.data	2036	2036
			.text	601296	601296
	lock	k32w061+release	(read/write)	691892	691892
			.bss	78704	78704
			.data	1996	1996
			.text	605392	605392
linux	chip-tool-no-interactive-ipv6only	arm64	(read only)	10283492	10283492
			(read/write)	492273	492273
			.bss	41025	41025
			.data	1168	1168
			.data.rel.ro	388840	388840
			.dynamic	560	560
			.got	57440	57440
			.init	24	24
			.init_array	184	184
			.rodata	513028	513028
			.text	8667044	8667044
	thermostat-no-ble	arm64	(read only)	2353620	2353620
			(read/write)	151121	151121
			.bss	63153	63153
			.data	1424	1424
			.data.rel.ro	78752	78752
			.dynamic	560	560
			.got	4768	4768
			.init	24	24
			.init_array	368	368
			.rodata	144548	144548
			.text	1980496	1980496
mbed	lock-app	CY8CPROTO_062_4343W+release	(read only)	6224	6224
			(read/write)	2368516	2368516
			.bss	185236	185236
			.data	5840	5840
			.text	`1331116`	`1331116`
nrfconnect	all-clusters-app	nrf52840dk_nrf52840	(read/write)	1163459	1163459
			bss	136528	136528
			rodata	147364	147364
			text	800940	800940
p6	all-clusters-app	default	(read/write)	2515120	2515120
			.bss	118640	118640
			.data	2768	2768
			.text	1473384	1473384
	light-app	default	(read/write)	2415592	2415592
			.bss	112136	112136
			.data	2576	2576
			.text	1373856	1373856
	lock-app	default	(read/write)	2379160	2379160
			.bss	111880	111880
			.data	2536	2536
			.text	1337424	1337424
telink	lighting-app	tlsr9518adk80d	(read/write)	801516	801516
			bss	69988	69988
			noinit	40416	40416
			text	570132	570132

mrjerryjohns

This is great, thanks @andreilitvin for turning this around so quickly.

github-actions · 2022-04-13T18:56:21Z

PR #17341: Size comparison from 94e8197 to dc6c9a3

Increases (8 builds for efr32, esp32, mbed, nrfconnect, telink)

platform	target	config	section	`94e8197`	`dc6c9a3`	change	% change
efr32	lighting-app	BRD4161A	(read only)	908052	908068	16	0.0
			.text	908044	908060	16	0.0
		BRD4161A+rpc	(read only)	942420	942436	16	0.0
			.text	942412	942428	16	0.0
	window-app	BRD4161A	(read only)	844708	844724	16	0.0
			.text	844700	844716	16	0.0
esp32	all-clusters-app	c3devkit	(read only)	979972	979976	4	0.0
			.flash.text	979972	979976	4	0.0
		m5stack	(read only)	1035459	`1035487`	28	0.0
			.flash.text	1030075	1030103	28	0.0
mbed	lock-app	CY8CPROTO_062_4343W+release	(read/write)	2368516	2369412	896	0.0
			.text	`1331116`	1332012	896	0.1
nrfconnect	all-clusters-app	nrf52840dk_nrf52840	(read/write)	1163459	`1163491`	32	0.0
			text	800940	800968	28	0.0
telink	lighting-app	tlsr9518adk80d	(read/write)	801516	801524	8	0.0
			text	570132	570136	4	0.0

Decreases (4 builds for cc13x2_26x2, cyw30739, linux)

platform	target	config	section	`94e8197`	`dc6c9a3`	change	% change
cc13x2_26x2	pump-app	LP_CC2652R7	(read only)	648527	648519	-8	-0.0
			.text	572624	572616	-8	-0.0
cyw30739	lock	cyw930739m2evb_01	(read/write)	576646	576638	-8	-0.0
			.app_xip_area	484892	484884	-8	-0.0
linux	chip-tool-no-interactive-ipv6only	arm64	(read only)	10283492	10283348	-144	-0.0
			.text	8667044	8666900	-144	-0.0
	thermostat-no-ble	arm64	(read only)	2353620	`2353476`	-144	-0.0
			.text	1980496	`1980352`	-144	-0.0

Full report (22 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)

platform	target	config	section	`94e8197`	`dc6c9a3`	change	% change
cc13x2_26x2	lock-ftd	LP_CC2652R7	(read only)	640435	640435	0	0.0
			(read/write)	151212	151212	0	0.0
			.bss	74144	74144	0	0.0
			.data	3212	3212	0	0.0
			.rodata	80227	80227	0	0.0
			.text	559716	559716	0	0.0
	lock-mtd	LP_CC2652R7	(read only)	589171	589171	0	0.0
			(read/write)	146932	146932	0	0.0
			.bss	69864	69864	0	0.0
			.data	3212	3212	0	0.0
			.rodata	80107	80107	0	0.0
			.text	508572	508572	0	0.0
	pump-app	LP_CC2652R7	(read only)	648527	648519	-8	-0.0
			(read/write)	152508	152508	0	0.0
			.bss	74640	74640	0	0.0
			.data	3244	3244	0	0.0
			.rodata	75415	75415	0	0.0
			.text	572624	572616	-8	-0.0
	pump-controller-app	LP_CC2652R7	(read only)	642499	642499	0	0.0
			(read/write)	152176	152176	0	0.0
			.bss	74344	74344	0	0.0
			.data	3208	3208	0	0.0
			.rodata	79051	79051	0	0.0
			.text	562960	562960	0	0.0
cyw30739	light	cyw930739m2evb_01	(read/write)	619042	619042	0	0.0
			.app_xip_area	525752	525752	0	0.0
			.bss	75956	75956	0	0.0
			.data	684	684	0	0.0
			.rodata	0	0	0	0.0
			.text	0	0	0	0.0
	lock	cyw930739m2evb_01	(read/write)	576646	576638	-8	-0.0
			.app_xip_area	484892	484884	-8	-0.0
			.bss	74452	74452	0	0.0
			.data	648	648	0	0.0
			.rodata	0	0	0	0.0
			.text	0	0	0	0.0
	ota-requestor-no-progress-logging	cyw930739m2evb_01	(read/write)	565170	565170	0	0.0
			.app_xip_area	463788	463788	0	0.0
			.bss	83784	83784	0	0.0
			.data	564	564	0	0.0
			.rodata	0	0	0	0.0
			.text	112	112	0	0.0
efr32	lighting-app	BRD4161A	(read only)	908052	908068	16	0.0
			(read/write)	133144	133144	0	0.0
			.bss	131104	131104	0	0.0
			.data	2040	2040	0	0.0
			.text	908044	908060	16	0.0
		BRD4161A+rpc	(read only)	942420	942436	16	0.0
			(read/write)	149828	149828	0	0.0
			.bss	147584	147584	0	0.0
			.data	2244	2244	0	0.0
			.text	942412	942428	16	0.0
	window-app	BRD4161A	(read only)	844708	844724	16	0.0
			(read/write)	131148	131148	0	0.0
			.bss	129200	129200	0	0.0
			.data	1948	1948	0	0.0
			.text	844700	844716	16	0.0
esp32	all-clusters-app	c3devkit	(read only)	979972	979976	4	0.0
			(read/write)	`1397578`	`1397578`	0	0.0
			.dram0.bss	62624	62624	0	0.0
			.dram0.data	14420	14420	0	0.0
			.flash.rodata	201616	201616	0	0.0
			.flash.text	979972	979976	4	0.0
			.iram0.text	62016	62016	0	0.0
		m5stack	(read only)	1035459	`1035487`	28	0.0
			(read/write)	465316	465316	0	0.0
			.dram0.bss	68144	68144	0	0.0
			.dram0.data	34152	34152	0	0.0
			.flash.rodata	231184	231184	0	0.0
			.flash.text	1030075	1030103	28	0.0
			.iram0.text	123107	123107	0	0.0
k32w	light	k32w061+release	(read/write)	687260	687260	0	0.0
			.bss	78128	78128	0	0.0
			.data	2036	2036	0	0.0
			.text	601296	601296	0	0.0
	lock	k32w061+release	(read/write)	691892	691892	0	0.0
			.bss	78704	78704	0	0.0
			.data	1996	1996	0	0.0
			.text	605392	605392	0	0.0
linux	chip-tool-no-interactive-ipv6only	arm64	(read only)	10283492	10283348	-144	-0.0
			(read/write)	492273	492273	0	0.0
			.bss	41025	41025	0	0.0
			.data	1168	1168	0	0.0
			.data.rel.ro	388840	388840	0	0.0
			.dynamic	560	560	0	0.0
			.got	57440	57440	0	0.0
			.init	24	24	0	0.0
			.init_array	184	184	0	0.0
			.rodata	513028	513028	0	0.0
			.text	8667044	8666900	-144	-0.0
	thermostat-no-ble	arm64	(read only)	2353620	`2353476`	-144	-0.0
			(read/write)	151121	151121	0	0.0
			.bss	63153	63153	0	0.0
			.data	1424	1424	0	0.0
			.data.rel.ro	78752	78752	0	0.0
			.dynamic	560	560	0	0.0
			.got	4768	4768	0	0.0
			.init	24	24	0	0.0
			.init_array	368	368	0	0.0
			.rodata	144548	144548	0	0.0
			.text	1980496	`1980352`	-144	-0.0
mbed	lock-app	CY8CPROTO_062_4343W+release	(read only)	6224	6224	0	0.0
			(read/write)	2368516	2369412	896	0.0
			.bss	185236	185236	0	0.0
			.data	5840	5840	0	0.0
			.text	`1331116`	1332012	896	0.1
nrfconnect	all-clusters-app	nrf52840dk_nrf52840	(read/write)	1163459	`1163491`	32	0.0
			bss	136528	136528	0	0.0
			rodata	147364	147364	0	0.0
			text	800940	800968	28	0.0
p6	all-clusters-app	default	(read/write)	2515120	2515120	0	0.0
			.bss	118640	118640	0	0.0
			.data	2768	2768	0	0.0
			.text	1473384	1473384	0	0.0
	light-app	default	(read/write)	2415592	2415592	0	0.0
			.bss	112136	112136	0	0.0
			.data	2576	2576	0	0.0
			.text	1373856	1373856	0	0.0
	lock-app	default	(read/write)	2379160	2379160	0	0.0
			.bss	111880	111880	0	0.0
			.data	2536	2536	0	0.0
			.text	1337424	1337424	0	0.0
telink	lighting-app	tlsr9518adk80d	(read/write)	801516	801524	8	0.0
			bss	69988	69988	0	0.0
			noinit	40416	40416	0	0.0
			text	570132	570136	4	0.0

andy31415 added 3 commits April 13, 2022 12:35

Add python build support into build_examples.py

db64a77

Ensure CI will also build python

f449cab

Updated unit tests

a87f565

boring-cyborg bot added github scripts workflows labels Apr 13, 2022

pullapprove bot requested review from selissia, tecimovic, saurabhst, turon, vijs, vivien-apple, wbschiller, woody-apple and xylophone21 April 13, 2022 16:43

pullapprove bot added the review - pending label Apr 13, 2022

pullapprove bot requested review from yufengwangca, woody-apple, yunhanw-google and xylophone21 April 13, 2022 16:43

Restyle

dc6c9a3

mrjerryjohns approved these changes Apr 13, 2022

View reviewed changes

bzbarsky-apple approved these changes Apr 14, 2022

View reviewed changes

pullapprove bot added review - approved and removed review - pending labels Apr 14, 2022

andy31415 merged commit ea4c3dd into project-chip:master Apr 14, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add python building build capabilities to build_examples.py #17341

Add python building build capabilities to build_examples.py #17341

andy31415 commented Apr 13, 2022

github-actions bot commented Apr 13, 2022 •

edited

Loading

mrjerryjohns left a comment

github-actions bot commented Apr 13, 2022 •

edited

Loading

Add python building build capabilities to build_examples.py #17341

Add python building build capabilities to build_examples.py #17341

Conversation

andy31415 commented Apr 13, 2022

Problem

Change overview

Testing

github-actions bot commented Apr 13, 2022 • edited Loading

mrjerryjohns left a comment

Choose a reason for hiding this comment

github-actions bot commented Apr 13, 2022 • edited Loading

github-actions bot commented Apr 13, 2022 •

edited

Loading

github-actions bot commented Apr 13, 2022 •

edited

Loading