Add a linux contact sensor #25161

andy31415 · 2023-02-17T17:42:51Z

Add a linux compilation for the contact sensor.
The control can be done via UI. I have built it using:

./scripts/build/build_examples.py --target linux-x64-contact-sensor-libnl-no-ble-with-ui build

Includes adding a UI window for boolean states and UI-based control of the state value.

examples/common/imgui_ui/windows/boolean_state.cpp

examples/contact-sensor-app/linux/main.cpp

github-actions · 2023-02-17T18:56:53Z

PR #25161: Size comparison from 0edb83a to 0233a06

Increases (1 build for cc32xx)

platform	target	config	section	`0edb83a`	`0233a06`	change	% change
cc32xx	lock	CC3235SF_LAUNCHXL	.debug_info	20361460	20361461	1	0.0

Full report (1 build for cc32xx)

platform	target	config	section	`0edb83a`	`0233a06`	change
cc32xx	lock	CC3235SF_LAUNCHXL		0	0	0
			(read only)	642217	642217	0
			(read/write)	203696	203696	0
			.ARM.attributes	44	44	0
			.ARM.exidx	8	8	0
			.bss	197096	197096	0
			.comment	194	194	0
			.data	1480	1480	0
			.debug_abbrev	934242	934242	0
			.debug_aranges	87472	87472	0
			.debug_frame	300288	300288	0
			.debug_info	20361460	20361461	1
			.debug_line	`2661258`	`2661258`	0
			.debug_loc	2797585	2797585	0
			.debug_ranges	281768	281768	0
			.debug_str	`3020435`	`3020435`	0
			.ramVecs	780	780	0
			.resetVecs	64	64	0
			.rodata	105705	105705	0
			.shstrtab	232	232	0
			.stab	204	204	0
			.stabstr	441	441	0
			.stack	2048	2048	0
			.strtab	377379	377379	0
			.symtab	256048	256048	0
			.text	534392	534392	0

github-actions · 2023-02-21T17:16:56Z

PR #25161: Size comparison from c56a2a7 to 60581d6

Decreases (1 build for cc32xx)

platform	target	config	section	`c56a2a7`	`60581d6`	change	% change
cc32xx	lock	CC3235SF_LAUNCHXL	.debug_info	20254060	20254059	-1	-0.0

Full report (1 build for cc32xx)

platform	target	config	section	`c56a2a7`	`60581d6`	change	% change
cc32xx	lock	CC3235SF_LAUNCHXL		0	0	0	0.0
			(read only)	642545	642545	0	0.0
			(read/write)	203664	203664	0	0.0
			.ARM.attributes	44	44	0	0.0
			.ARM.exidx	8	8	0	0.0
			.bss	197064	197064	0	0.0
			.comment	194	194	0	0.0
			.data	1480	1480	0	0.0
			.debug_abbrev	930168	930168	0	0.0
			.debug_aranges	87272	87272	0	0.0
			.debug_frame	299752	299752	0	0.0
			.debug_info	20254060	20254059	-1	-0.0
			.debug_line	2655143	2655143	0	0.0
			.debug_loc	2795383	2795383	0	0.0
			.debug_ranges	281592	281592	0	0.0
			.debug_str	`3017470`	`3017470`	0	0.0
			.ramVecs	780	780	0	0.0
			.resetVecs	64	64	0	0.0
			.rodata	105817	105817	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	377598	377598	0	0.0
			.symtab	256144	256144	0	0.0
			.text	534604	534604	0	0.0

…mplex unfortunately

examples/common/imgui_ui/windows/boolean_state.cpp

examples/contact-sensor-app/linux/main.cpp

github-actions · 2023-02-21T18:28:55Z

PR #25161: Size comparison from c56a2a7 to bf69b1f

Decreases (1 build for cc32xx)

platform	target	config	section	`c56a2a7`	`bf69b1f`	change	% change
cc32xx	lock	CC3235SF_LAUNCHXL	.debug_info	20254060	20254059	-1	-0.0

Full report (2 builds for cc32xx, mbed)

platform	target	config	section	`c56a2a7`	`bf69b1f`	change	% change
cc32xx	lock	CC3235SF_LAUNCHXL		0	0	0	0.0
			(read only)	642545	642545	0	0.0
			(read/write)	203664	203664	0	0.0
			.ARM.attributes	44	44	0	0.0
			.ARM.exidx	8	8	0	0.0
			.bss	197064	197064	0	0.0
			.comment	194	194	0	0.0
			.data	1480	1480	0	0.0
			.debug_abbrev	930168	930168	0	0.0
			.debug_aranges	87272	87272	0	0.0
			.debug_frame	299752	299752	0	0.0
			.debug_info	20254060	20254059	-1	-0.0
			.debug_line	2655143	2655143	0	0.0
			.debug_loc	2795383	2795383	0	0.0
			.debug_ranges	281592	281592	0	0.0
			.debug_str	`3017470`	`3017470`	0	0.0
			.ramVecs	780	780	0	0.0
			.resetVecs	64	64	0	0.0
			.rodata	105817	105817	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	377598	377598	0	0.0
			.symtab	256144	256144	0	0.0
			.text	534604	534604	0	0.0
mbed	lock-app	CY8CPROTO_062_4343W+release	(read only)	6224	6224	0	0.0
			(read/write)	2466784	2466784	0	0.0
			.bss	215780	215780	0	0.0
			.data	5880	5880	0	0.0
			.text	1429428	1429428	0	0.0

github-actions · 2023-02-21T19:05:09Z

PR #25161: Size comparison from c56a2a7 to c7340a8

Full report (1 build for cc32xx)

platform	target	config	section	`c56a2a7`	`c7340a8`
cc32xx	lock	CC3235SF_LAUNCHXL		0	0
			(read only)	642545	642545
			(read/write)	203664	203664
			.ARM.attributes	44	44
			.ARM.exidx	8	8
			.bss	197064	197064
			.comment	194	194
			.data	1480	1480
			.debug_abbrev	930168	930168
			.debug_aranges	87272	87272
			.debug_frame	299752	299752
			.debug_info	20254060	20254060
			.debug_line	2655143	2655143
			.debug_loc	2795383	2795383
			.debug_ranges	281592	281592
			.debug_str	`3017470`	`3017470`
			.ramVecs	780	780
			.resetVecs	64	64
			.rodata	105817	105817
			.shstrtab	232	232
			.stab	204	204
			.stabstr	441	441
			.stack	2048	2048
			.strtab	377598	377598
			.symtab	256144	256144
			.text	534604	534604

* A linux contact sensor app * Fix project config * Also add occupancy sensing UI * Restyle * Added TODO for boolean state - no impl yet though * Remove unnedded init override * Copy over networking logic form all clusters. This seems unusually complex unfortunately * Use bridge as an example of network commissioning * Update HAL comment with linking the boolean state issue * Restyle * Ensure port parameters are enabled in the contact sensor app --------- Co-authored-by: Andrei Litvin <[email protected]>

A linux contact sensor app

ad33d28

andy31415 force-pushed the linux_sensor branch from bc5938d to ad33d28 Compare February 17, 2023 17:44

andy31415 added 3 commits February 17, 2023 12:46

Fix project config

bb6df6f

Also add occupancy sensing UI

d3fa403

Restyle

0233a06

pullapprove bot requested review from turon, vijs, vivien-apple, woody-apple, xylophone21, yufengwangca and yunhanw-google February 17, 2023 18:15

github-actions bot added examples scripts labels Feb 17, 2023

bzbarsky-apple approved these changes Feb 17, 2023

View reviewed changes

andreilitvin added 3 commits February 21, 2023 11:29

Merge branch 'master' into linux_sensor

dc0a946

Added TODO for boolean state - no impl yet though

3deba08

Remove unnedded init override

60581d6

Copy over networking logic form all clusters. This seems unusually co…

a3168fc

…mplex unfortunately

bzbarsky-apple requested changes Feb 21, 2023

View reviewed changes

examples/common/imgui_ui/windows/boolean_state.cpp Show resolved Hide resolved

examples/contact-sensor-app/linux/main.cpp Outdated Show resolved Hide resolved

pullapprove bot requested a review from nivi-apple February 21, 2023 18:01

andreilitvin added 3 commits February 21, 2023 13:06

Use bridge as an example of network commissioning

0924fe8

Update HAL comment with linking the boolean state issue

05ae0be

Restyle

bf69b1f

Ensure port parameters are enabled in the contact sensor app

c7340a8

bzbarsky-apple approved these changes Feb 21, 2023

View reviewed changes

tcarmelveilleux approved these changes Feb 21, 2023

View reviewed changes

pullapprove bot added review - approved and removed review - pending labels Feb 21, 2023

andy31415 merged commit 8d4fbed into project-chip:master Feb 21, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add a linux contact sensor #25161

Add a linux contact sensor #25161

andy31415 commented Feb 17, 2023 •

edited

Loading

github-actions bot commented Feb 17, 2023

github-actions bot commented Feb 21, 2023

github-actions bot commented Feb 21, 2023

github-actions bot commented Feb 21, 2023

Add a linux contact sensor #25161

Add a linux contact sensor #25161

Conversation

andy31415 commented Feb 17, 2023 • edited Loading

github-actions bot commented Feb 17, 2023

github-actions bot commented Feb 21, 2023

github-actions bot commented Feb 21, 2023

github-actions bot commented Feb 21, 2023

andy31415 commented Feb 17, 2023 •

edited

Loading