Skip to content

Conversation

@BujSet
Copy link
Contributor

@BujSet BujSet commented Aug 7, 2025

Summary

Previously, only the example Add model was included in the matrix of models that the Zephyr CI job tests. The Add model isn't very representative of real-world models, and may give a false sense of security on the functionality of incorporating ExecuTorch with Zephyr. This PR adds the simple Softmax and Mobilenet V2 models to the test matrix. While the Softmax model is like the Add model (a toy example), Mobilenet V2 includes using an image from the imagenet validation set, and verifies that the top predicted label matches expectation (i.e. predict "Samoyed" as best label). All three of these tests show example of this flow can be extend further for other models. Thus this PR integrates these test to ensure compatibility between ExecuTorch and Zephyr is maintained.

Test plan

Verified that these models work with the Zephyr ecosystem manually in a separate docker image.

@pytorch-bot
Copy link

pytorch-bot bot commented Aug 7, 2025

🔗 Helpful Links

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

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

❌ 3 New Failures, 19 Pending, 1 Unrelated Failure

As of commit b2a355e with merge base f7f486d (image):

NEW FAILURES - The following jobs have failed:

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.

@meta-cla 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 Aug 7, 2025
@BujSet BujSet self-assigned this Aug 7, 2025
@BujSet BujSet added ciflow/trunk module: microcontrollers For embedded MCUs like Cortex-M, or RTOS like Zephyr, does not track NPU backend like Arm Ethos. labels Aug 7, 2025
@BujSet
Copy link
Contributor Author

BujSet commented Aug 7, 2025

@pytorchbot label "release notes: none"

@BujSet BujSet marked this pull request as ready for review August 7, 2025 17:38
@pytorch-bot pytorch-bot bot added the release notes: none Do not include this in the release notes label Aug 7, 2025
@BujSet BujSet force-pushed the zephyr_ci_models_test_more_models branch from 32857db to 561dd84 Compare August 7, 2025 18:08
@BujSet BujSet requested a review from kirklandsign as a code owner August 7, 2025 18:08
@BujSet BujSet force-pushed the zephyr_ci_models_test_more_models branch 6 times, most recently from 3980a1f to d99348c Compare August 8, 2025 02:36
@BujSet BujSet force-pushed the zephyr_ci_models_test_more_models branch from d99348c to b2a355e Compare August 8, 2025 04:06
@BujSet BujSet merged commit be221c6 into pytorch:main Aug 8, 2025
229 of 234 checks passed
@BujSet BujSet deleted the zephyr_ci_models_test_more_models branch August 8, 2025 04:46
agrima1304 pushed a commit to agrima1304/executorch that referenced this pull request Aug 26, 2025
### Summary
Previously, only the example Add model was included in the matrix of
models that the Zephyr CI job tests. The Add model isn't very
representative of real-world models, and may give a false sense of
security on the functionality of incorporating ExecuTorch with Zephyr.
This PR adds the simple `Softmax` and `Mobilenet V2` models to the test
matrix. While the `Softmax` model is like the `Add` model (a toy
example), `Mobilenet V2` includes using an image from the `imagenet`
validation set, and verifies that the top predicted label matches
expectation (i.e. predict "Samoyed" as best label). All three of these
tests show example of this flow can be extend further for other models.
Thus this PR integrates these test to ensure compatibility between
ExecuTorch and Zephyr is maintained.

### Test plan
Verified that these models work with the Zephyr ecosystem manually in a
separate docker image.
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: 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.

2 participants