Mlsw 10552 setup ci to build and test machine learning applications repo #5

silabs-JeevanRH · 2025-10-25T15:30:53Z

Description

This PR is to setup CI to build and test machine_learning_applications repo apps

Type of change

Please select all the options that apply. Ideally only one should be selected.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
This change requires a documentation update

How Has This Been Tested?

The goal of this task was to verify that all machine learning applications in the repository build successfully whenever any push is made. Since the CI pipeline was newly created, it was tested by pushing the updated workflow code to the repository, triggering the GitHub Actions job, and confirming that all matrix builds executed successfully end-to-end without errors, ensuring the pipeline correctly generates and compiles every ML application automatically.

Test Configuration:

Firmware version: N/A
Hardware: BRD2601B
Toolchain: ARM GNU GCC 12.2.rel1
SDKs: Simplicity sdk v2025.6.2, AI/ML extension v2.1.2

Screenshots

Checklist

My code follows the style guidelines of this project
I have performed a self-review of my code
I have commented my code, particularly in hard-to-understand areas
I have made corresponding changes to the documentation
My changes generate no new warnings
I have added tests that prove my fix is effective or that my feature works
New and existing unit tests pass locally with my changes
Any dependent changes have been merged and published in downstream modules

silabs-raashid

Some work is needed on the CI job, please address those comments and we should be good to go. Marking this as "Request Changes" for now.

silabs-raashid · 2025-10-29T15:23:43Z

.github/workflows/CI.yml

please change the file name to lowercase ci.yml

file name is still in uppercase (CI.yml). Please make it lowercase (ci.yml).

.github/workflows/CI.yml

silabs-raashid · 2025-10-29T15:34:29Z

.github/workflows/CI.yml

@@ -0,0 +1,119 @@
+name: Ubuntu env 
+on: [push]


This is good until you're testing the pipeline. After which we should make the condition more stringent. My suggestion is to have the following:

Suggested change

on: [push]

on:

push:

branches:

- main

- feature/*

tags:

- v*

pull_request:

branches:

- main

- feature/*

workflow_dispatch:

This should nail down this workflow to run only when some pushes code to main or a feature/* branch. Also, this workflow will only run when a PR is created against main or a feature/* branch. Lastly, the workflow_dispatch will allow us to run this job on-demand from the UI.

silabs-raashid · 2025-10-29T16:14:11Z

.github/workflows/CI.yml

+        board: [brd2601b]
+        app:
+          - imu/ble_magic_wand/ble_magic_wand
+          - imu/imu_anomaly_detection/imu_anomaly_detection
+          - vision/people_flow_counter_mlx90640/people_flow_counter_mlx90640
+          - voice/sensory_wakeupword/app/sensory_wakeupword_series_1
+          - voice/sensory_wakeupword/app/sensory_wakeupword_series_2


This solution is a good first pass. However, we need to have a more robust solution later on. This implementation assumes that every app in the app array work on every board in the board array. However, this is certainly not going to be the case. I can see that templates.xml file has the "sensory_wakeupword" project that is only compatible with "brd4166a" and we completely missing that here.

My suggestion would be to extract the path to the .slcp files from the projectFilePaths key and the supported boards from the boardCompatibility key in the templates.xml file.

.github/workflows/CI.yml

silabs-raashid · 2025-10-29T16:23:08Z

.github/workflows/CI.yml

+  contents: read
+
+jobs:
+  prepare-sdk-and-tools:


try splitting the checkout in to a separate job and tools download in another since they are not dependent on each other and can run in parallel in that case.

silabs-raashid · 2025-10-29T16:24:20Z

.github/workflows/CI.yml

+      - name: Upload prepared sources
+        uses: actions/upload-artifact@v5
+        with:
+          name: gsdk-and-tools
+          path: |
+            src/
+            tools/
+          include-hidden-files: true


Once you split the checkout and tools download jobs, you can upload the artifacts separately. Just make sure to give them different names.

silabs-raashid · 2025-10-29T16:24:50Z

.github/workflows/CI.yml

+      - name: Download prepared sources
+        uses: actions/download-artifact@v4
+        with:
+          name: gsdk-and-tools
+          path: .


After splitting the jobs, you'll need to download them in two separate steps. For one download the path will be src/ and for the other it will be tools/

silabs-raashid · 2025-10-29T16:25:32Z

.github/workflows/CI.yml

+        run: |
+          set -e
+          slc configuration --sdk "${{ github.workspace }}/src/gsdk"
+          slc signature trust --sdk     "${{ github.workspace }}/src/gsdk"


please remove random spacing.

Co-authored-by: Raashid Ansari <raashid.ansari@silabs.com>

…ing and downloading sdk and tools separtely

…YAML upload indentation

…tion

silabs-raashid

A few minor changes needed again. But we're close to the finish line.

silabs-raashid · 2025-10-31T17:24:38Z

.github/workflows/CI.yml

file name is still in uppercase (CI.yml). Please make it lowercase (ci.yml).

silabs-raashid · 2025-10-31T17:26:20Z

.github/workflows/CI.yml

+      - name: Upload SDK
+        uses: actions/upload-artifact@v5
+        with:
+          name: SDK


Suggested change

name: SDK

name: sisdk-and-extensions

silabs-raashid · 2025-10-31T17:26:55Z

.github/workflows/CI.yml

+      - name: Upload Tools
+        uses: actions/upload-artifact@v5
+        with:
+          name: ARM_GNU-and-slc-toolchain


Suggested change

name: ARM_GNU-and-slc-toolchain

name: arm-gnu-toolchain-and-slc

silabs-raashid · 2025-10-31T17:27:33Z

.github/workflows/CI.yml

+      - name: Download GSDK
+        uses: actions/download-artifact@v5
+        with:
+          name: SDK


Suggested change

name: SDK

name: sisdk-and-extensions

silabs-raashid · 2025-10-31T17:29:04Z

.github/workflows/CI.yml

+          name: ARM_GNU-and-slc-toolchain
+          path: .


Suggested change

name: ARM_GNU-and-slc-toolchain

path: .

name: arm-gnu-toolchain-and-slc

path: .

silabs-raashid · 2025-10-31T17:32:51Z

.github/workflows/CI.yml

+          path: .
+          include-hidden-files: true
+
+  build-ml:


Suggested change

build-ml:

build:

silabs-raashid · 2025-10-31T17:35:24Z

Also, please resolve the conflict that this file, people_flow_counter_mls90460.slcp

Jeevan Revaneppa Hirethanad added 30 commits October 21, 2025 12:26

Add CI workflow

e80cfa3

update CI

821f3b6

update CI OS

f595544

update CI new

a3e20a8

update CI OSs

5261098

update CI newm

f9bb9a4

update CI neww

f16b11a

update CI neww

38fa9bc

update CI newupadtes

c8b025f

update CI newwww

a0cb5d4

update CI newweded

884d629

update CI latest1137

237b37a

update CI latest1205

99ca1f4

update CI latest1226

af60f0d

update CI latest1243

2964e02

update CI latest0109

b725bf5

update CI latest0132

efd9aa9

update CI latest0202

233399f

update CI latest0232

192cebd

update CI latest0240

e8c400b

update CI latest0456

a24c4c3

update CI latest512

2cfd993

update CI latest0520

38f455a

update CI latest0531

f110ab8

update CI latest957

c7031ea

update CI latest1013

785633e

update CI latest1020

04d120d

update CI latest1024

1fd5446

update CI latest1035

eccc103

update CI latest1050

e2040df

Jeevan Revaneppa Hirethanad added 5 commits October 25, 2025 16:55

Latest CI 239

dcae63f

Latest CI 2395

dd22856

Latest CI 2395

bb33727

update CI latest335

58c4d2f

update CI latest335

c90dc7b

silabs-raashid requested changes Oct 29, 2025

View reviewed changes

silabs-JeevanRH and others added 22 commits October 29, 2025 22:10

Update .github/workflows/CI.yml

3334974

Co-authored-by: Raashid Ansari <raashid.ansari@silabs.com>

generate apps matrix dynamically based on templates.xml, split upload…

831bf62

…ing and downloading sdk and tools separtely

Fix genarte_build.py filename issues

23a6dc5

Fix genarte_build.py filename issues

27de542

Fix genarte_build.py building issues

8d3a128

Fix genarte_build.py filename issues

d63e450

Fix genarte_build.py filename issues

2360778

Update template.xml path in genarte_build.py

d81dc29

ensure generate_matrix.py outputs valid JSON for fromJSON() and fix …

1c74b6d

…YAML upload indentation

removw unwanted print statement in genarte_build.py

06550d8

ensure generate_matrix.py outputs valid JSON for fromJSON() and fix …

d5d6e66

…YAML upload indentation

ensure generate_matrix.py outputs valid JSON for fromJSON() and fix …

ba88781

…YAML upload indentation

debug file structure

1e4b3d2

update file structure

8c3a617

Update Board compatibility

94b7cf4

Remove unsupported example from template xml and add CI trigger condi…

1f61128

…tion

CI trigger on push

2443f30

CI trigger on push

f7ed6ba

CI trigger condition update

0272ca3

Retrigger CI to see if json modification works

fc515da

CI trigger on push

932d25d

CI trigger condition update

c01f337

silabs-raashid requested changes Oct 31, 2025

View reviewed changes

-on: [push]
+on:
+  push:
+    branches:
+      - main
+      - feature/*
+    tags:
+      - v*
+  pull_request:
+    branches:
+      - main
+      - feature/*
+  workflow_dispatch:

	name: ARM_GNU-and-slc-toolchain
	name: arm-gnu-toolchain-and-slc

Mlsw 10552 setup ci to build and test machine learning applications repo #5

Are you sure you want to change the base?

Mlsw 10552 setup ci to build and test machine learning applications repo #5

Uh oh!

Conversation

silabs-JeevanRH commented Oct 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Type of change

How Has This Been Tested?

Screenshots

Checklist

Uh oh!

silabs-raashid left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

silabs-raashid left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

silabs-raashid commented Oct 31, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

silabs-JeevanRH commented Oct 25, 2025 •

edited

Loading