Add Pico2 build validation CI (trunk only) by psiddh · Pull Request #18900 · pytorch/executorch

psiddh · 2026-04-15T00:07:29Z

Adds two parallel CI jobs that validate FP32 and CMSIS-NN INT8 Pico2 firmware builds on every push to main. Each job exports a model, cross- compiles firmware, and validates the .elf/.uf2 artifacts fit within Pico2 memory limits (520KB SRAM, 4MB flash).

Triggered on push to main when examples/raspberry_pi/pico2/** or backends/cortex_m/** files change, plus a daily schedule and release branches

Adds two parallel CI jobs that validate FP32 and CMSIS-NN INT8 Pico2 firmware builds on every push to main. Each job exports a model, cross- compiles firmware, and validates the .elf/.uf2 artifacts fit within Pico2 memory limits (520KB SRAM, 4MB flash). Triggered on push to main when examples/raspberry_pi/pico2/** or backends/cortex_m/** files change, plus a daily schedule. Co-authored-by: Claude <noreply@anthropic.com>

pytorch-bot · 2026-04-15T00:07:32Z

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/18900

📄 Preview Python docs built from this PR

Note: Links to docs will display an error until the docs builds have been completed.

❌ 3 New Failures, 2 Pending, 1 Unrelated Failure

As of commit 0ee15b8 with merge base 26e2ab8 ():

NEW FAILURES - The following jobs have failed:

trunk / test-torchao-huggingface-checkpoints (lfm2_5_1_2b, linux.arm64.2xlarge, executorch-ubuntu-22.04-g... / linux-job (gh)
RuntimeError: Command docker exec -t 8c5c2bd2a20ebd1f1cbe3ea05f2bbf98f2d9106729e627ac88499e68201f461d /exec failed with exit code 1
trunk / test-torchao-huggingface-checkpoints (phi_4_mini, linux.arm64.2xlarge, executorch-ubuntu-22.04-gc... / linux-job (gh)
RuntimeError: Command docker exec -t 7fdbd5cfc562d1d2117c52040764a4a39c849363093ce509d8bcbe5d6959d503 /exec failed with exit code 1
trunk / test-torchao-huggingface-checkpoints (qwen3_4b, linux.arm64.2xlarge, executorch-ubuntu-22.04-gcc1... / linux-job (gh)
RuntimeError: Command docker exec -t f47c182a8b51ff0759feb59ac038d9dfa56a1c91606e2aaeb6e0096ff3b99e4e /exec failed with exit code 1

BROKEN TRUNK - The following job failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

trunk / unittest-release / windows / windows-job (gh) (trunk failure)
##[error]The operation was canceled.

This comment was automatically generated by Dr. CI and updates every 15 minutes.

github-actions · 2026-04-15T00:08:11Z

This PR needs a `release notes:` label

If your change should be included in the release notes (i.e. would users of this library care about this change?), please use a label starting with release notes:. This helps us keep track and include your important work in the next release notes.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "release notes: none"

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

Copilot

Pull request overview

Adds a new GitHub Actions workflow to continuously validate Raspberry Pi Pico2 firmware builds (FP32 and CMSIS-NN INT8), including artifact existence checks and basic memory-size guardrails, to catch regressions from changes in the Pico2 example or Cortex-M backend.

Changes:

Introduces a new test-pico2-build.yml workflow with two parallel jobs (FP32 + CMSIS-NN).
Each job exports a model, cross-compiles Pico2 firmware, and checks for expected .elf/.uf2 artifacts plus memory-size constraints.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

- Use arm-none-eabi-size aggregated totals (text+data for flash, data+bss for SRAM) instead of brittle per-section awk parsing - Increase timeout from 60 to 120 minutes to match other ARM SDK jobs - Add event_name to concurrency group to prevent cross-event cancellation - Add .ci/scripts/** and examples/arm/** to path triggers - CMSIS-NN symbol check now fails the job instead of warning Co-authored-by: Claude <noreply@anthropic.com>

Temporary: allows testing the workflow on this PR. Remove after validation.

Copilot

Pull request overview

Adds a new GitHub Actions workflow to continuously validate Raspberry Pi Pico2 firmware builds (FP32 and CMSIS-NN INT8) and enforce memory-size constraints, helping prevent upstream changes from breaking embedded builds.

Changes:

Introduces .github/workflows/test-pico2-build.yml with two parallel CI jobs for FP32 and CMSIS-NN Pico2 firmware builds.
Each job exports a model, cross-compiles firmware, and validates .elf/.uf2 artifacts plus SRAM/flash usage thresholds.
CMSIS-NN job additionally checks that CMSIS-NN-related symbols are present in the linked binary.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

CMSIS-NN models contain only cortex_m:: ops which are registered by the cortex_m backend, not by portable kernel codegen. Passing these to EXECUTORCH_SELECT_OPS_MODEL produces an empty #include "" in the generated registration code, failing the build. Skip SELECT_OPS_FLAGS when --cmsis is used. Co-authored-by: Claude <noreply@anthropic.com>

- Remove pull_request trigger (testing complete, both jobs passed) - Use head.sha for PR runs to match repo convention Co-authored-by: Claude <noreply@anthropic.com>

Copilot

Pull request overview

Adds dedicated CI coverage for Raspberry Pi Pico2 firmware builds, ensuring both FP32 and CMSIS-NN INT8 configurations continue to cross-compile and fit within Pico2 memory constraints.

Changes:

Update Pico2 firmware build script to skip selective build when CMSIS-NN mode is enabled.
Add a new GitHub Actions workflow that exports models, builds Pico2 firmware (FP32 + CMSIS-NN), and validates produced artifacts and memory usage on push/schedule.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File	Description
examples/raspberry_pi/pico2/build_firmware_pico.sh	Skips selective-build flags when building CMSIS-NN firmware to avoid selective-build incompatibilities with cortex_m ops.
.github/workflows/test-pico2-build.yml	New CI workflow running two build-and-validate jobs (FP32 + CMSIS-NN) with artifact presence checks and flash/SRAM size validation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

- Add set -euo pipefail to both script blocks so pipeline failures in size validation are not masked - Add workflow file itself to push paths so edits trigger CI Co-authored-by: Claude <noreply@anthropic.com>

Copilot AI review requested due to automatic review settings April 15, 2026 00:07

meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 15, 2026

Copilot started reviewing on behalf of psiddh April 15, 2026 00:08 View session

psiddh requested review from AdrianLundell, digantdesai, mergennachin and rascani April 15, 2026 00:08

Copilot AI reviewed Apr 15, 2026

View reviewed changes

psiddh added the ciflow/trunk label Apr 15, 2026

psiddh and others added 2 commits April 14, 2026 17:52

Add pull_request trigger to validate CI workflow

a1f6544

Temporary: allows testing the workflow on this PR. Remove after validation.

Copilot AI review requested due to automatic review settings April 15, 2026 00:54

Copilot started reviewing on behalf of psiddh April 15, 2026 00:55 View session

Copilot AI reviewed Apr 15, 2026

View reviewed changes

Comment thread .github/workflows/test-pico2-build.yml

Comment thread .github/workflows/test-pico2-build.yml Outdated

Comment thread .github/workflows/test-pico2-build.yml Outdated

Comment thread .github/workflows/test-pico2-build.yml Outdated

psiddh and others added 2 commits April 15, 2026 09:21

Remove temporary pull_request trigger, fix ref for PR runs

664cd2d

- Remove pull_request trigger (testing complete, both jobs passed) - Use head.sha for PR runs to match repo convention Co-authored-by: Claude <noreply@anthropic.com>

Copilot AI review requested due to automatic review settings April 15, 2026 18:37

Copilot started reviewing on behalf of psiddh April 15, 2026 18:38 View session

psiddh mentioned this pull request Apr 15, 2026

Pico2 Improvements (MlSys Updates) #18908

Closed

Copilot AI reviewed Apr 15, 2026

View reviewed changes

Comment thread .github/workflows/test-pico2-build.yml

Comment thread .github/workflows/test-pico2-build.yml

Comment thread .github/workflows/test-pico2-build.yml

Comment thread .github/workflows/test-pico2-build.yml

Comment thread .github/workflows/test-pico2-build.yml

Add pipefail and self-trigger path

0ee15b8

- Add set -euo pipefail to both script blocks so pipeline failures in size validation are not masked - Add workflow file itself to push paths so edits trigger CI Co-authored-by: Claude <noreply@anthropic.com>

psiddh requested a review from GregoryComer April 15, 2026 20:11

GregoryComer approved these changes Apr 15, 2026

View reviewed changes

psiddh merged commit 28c56fe into pytorch:main Apr 15, 2026
413 of 422 checks passed

Conversation

psiddh commented Apr 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

pytorch-bot Bot commented Apr 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/18900

❌ 3 New Failures, 2 Pending, 1 Unrelated Failure

Uh oh!

github-actions Bot commented Apr 15, 2026

This PR needs a release notes: label

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

psiddh commented Apr 15, 2026 •

edited

Loading

pytorch-bot Bot commented Apr 15, 2026 •

edited

Loading

This PR needs a `release notes:` label