Skip to content

Conversation

@BujSet
Copy link
Contributor

@BujSet BujSet commented Jul 22, 2025

Summary

Adding in CI tests to verify and maintain support for Zephyr projects that build executorch as a module. The main zephyr application source code is hosted externally for now, but the process to host an Executorch as an official module of zephyr is in progress.

This PR runs the simple Add model as unit tests, and verifies that the output is 2.0 (1D tensor + 1D tensor, values filled with 1.0s). This is intended to be a simple and fast test to exemplify how to use executorch to build an executor_runner for other models.

Test plan

Added a new CI test called test-models-arm-zephyr that exits with status 0 on success, and 1 on failure.

@pytorch-bot
Copy link

pytorch-bot bot commented Jul 22, 2025

🔗 Helpful Links

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

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

⏳ 2 Pending, 1 Unrelated Failure

As of commit 5e1a389 with merge base fead7c7 (image):

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

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

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

@facebook-github-bot facebook-github-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 Jul 22, 2025
@BujSet BujSet self-assigned this Jul 22, 2025
@BujSet BujSet added module: ci Issues related to continuous integration ciflow/trunk ciflow/binaries/all Release PRs with this label will build wheels for all python versions labels Jul 22, 2025
@BujSet
Copy link
Contributor Author

BujSet commented Jul 22, 2025

@pytorchbot label "release notes: none"

@pytorch-bot pytorch-bot bot added the release notes: none Do not include this in the release notes label Jul 22, 2025
@BujSet BujSet force-pushed the zephyr_ci_add_test branch from 0dbaafe to 03b8695 Compare July 23, 2025 20:36
@BujSet BujSet removed the ciflow/binaries/all Release PRs with this label will build wheels for all python versions label Jul 23, 2025
BujSet added a commit that referenced this pull request Jul 23, 2025
### Summary
Previously the PAL file was uploaded as a placeholder for Zephyr apps to
use. However, building `arm_zephyr.o` object was unsuccessful. The PAL
has been updated and can now be used and linked to zephyr executorch
applications.

### Test plan
Tested on a simple add model arm executor runner, see
[here](https://github.com/BujSet/zephyr/blob/5801a783883f3cf9bb172213d891c0de596c3dcf/samples/modules/executorch/arm-fvp-tutorials/models/CMakeLists.txt#L61)
for a concrete example. CI tests are WIP, see
[here](#12734).
@BujSet BujSet force-pushed the zephyr_ci_add_test branch from 03b8695 to 1521bb6 Compare July 23, 2025 23:16
@BujSet BujSet force-pushed the zephyr_ci_add_test branch 4 times, most recently from 52b5bcb to 6220541 Compare July 24, 2025 22:25
Conarnar pushed a commit to Conarnar/executorch that referenced this pull request Jul 25, 2025
### Summary
Previously the PAL file was uploaded as a placeholder for Zephyr apps to
use. However, building `arm_zephyr.o` object was unsuccessful. The PAL
has been updated and can now be used and linked to zephyr executorch
applications.

### Test plan
Tested on a simple add model arm executor runner, see
[here](https://github.com/BujSet/zephyr/blob/5801a783883f3cf9bb172213d891c0de596c3dcf/samples/modules/executorch/arm-fvp-tutorials/models/CMakeLists.txt#L61)
for a concrete example. CI tests are WIP, see
[here](pytorch#12734).
@BujSet BujSet force-pushed the zephyr_ci_add_test branch 4 times, most recently from b0c1cbc to fe2dc9b Compare July 25, 2025 22:30
@BujSet BujSet added the module: microcontrollers For embedded MCUs like Cortex-M, or RTOS like Zephyr, does not track NPU backend like Arm Ethos. label Jul 28, 2025
@BujSet BujSet force-pushed the zephyr_ci_add_test branch 2 times, most recently from c244439 to 82f5381 Compare July 28, 2025 21:10
@BujSet BujSet removed the module: ci Issues related to continuous integration label Jul 28, 2025
@BujSet BujSet force-pushed the zephyr_ci_add_test branch from 82f5381 to 71a1983 Compare July 28, 2025 22:24
@BujSet BujSet marked this pull request as ready for review July 28, 2025 22:24
@BujSet BujSet added the module: ci Issues related to continuous integration label Jul 28, 2025
@BujSet BujSet force-pushed the zephyr_ci_add_test branch 2 times, most recently from ac0b406 to 8811a7e Compare July 29, 2025 03:30
BujSet added 19 commits July 29, 2025 05:27
…ctory, moving all zpehyr stuff into the et repo (not ideal)
…scratch dir for zephyr CI tests (again, not ideal)
@BujSet BujSet force-pushed the zephyr_ci_add_test branch from 8811a7e to 5e1a389 Compare July 29, 2025 05:27
@BujSet BujSet merged commit 154a259 into pytorch:main Jul 29, 2025
596 of 610 checks passed
@BujSet BujSet deleted the zephyr_ci_add_test branch July 29, 2025 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunk CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. module: ci Issues related to continuous integration module: microcontrollers For embedded MCUs like Cortex-M, or RTOS like Zephyr, does not track NPU backend like Arm Ethos. release notes: none Do not include this in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants