Skip to content

Conversation

@BujSet
Copy link
Contributor

@BujSet BujSet commented Aug 7, 2025

Summary

In the baseline version of this test, only operator selective build is used. A CMake command line option was created to pass in an exported model's PTE file via command line so that the model API for data type selective build can be used. Adding verbose error checking to make logs more informative.

Test plan

Ran with and without the flag set, and saw that enabling data type selective build reduces memory footprint. Specifically for the example add model, operator library shrinks from ~127KB to ~83KB.

@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/13191

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

❌ 3 New Failures, 1 Cancelled Job, 6 Pending, 3 Unrelated Failures

As of commit 8ab3b03 with merge base 5635941 (image):

NEW FAILURES - The following jobs have failed:

CANCELLED JOB - The following job was cancelled. Please retry:

FLAKY - The following job failed but was likely due to flakiness present on trunk:

BROKEN TRUNK - The following jobs failed but was 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"

@pytorch-bot pytorch-bot bot added the release notes: none Do not include this in the release notes label Aug 7, 2025
@BujSet BujSet marked this pull request as ready for review August 7, 2025 17:29
@BujSet BujSet requested a review from kirklandsign as a code owner August 7, 2025 18:08
@BujSet BujSet force-pushed the dtype_selective_build_for_zephyr_ci_add_test branch 2 times, most recently from a3a5893 to 48ddd07 Compare August 7, 2025 20:39
@BujSet BujSet force-pushed the dtype_selective_build_for_zephyr_ci_add_test branch from 2d16f92 to f2e1389 Compare August 8, 2025 02:01
@BujSet BujSet merged commit f7f486d into pytorch:main Aug 8, 2025
224 of 232 checks passed
@BujSet BujSet deleted the dtype_selective_build_for_zephyr_ci_add_test branch August 8, 2025 04:00
agrima1304 pushed a commit to agrima1304/executorch that referenced this pull request Aug 26, 2025
…13191)

### Summary
In the baseline version of this test, only operator selective build is
used. A CMake command line option was created to pass in an exported
model's PTE file via command line so that the model API for data type
selective build can be used. Adding verbose error checking to make logs
more informative.

### Test plan
Ran with and without the flag set, and saw that enabling data type
selective build reduces memory footprint. Specifically for the example
add model, operator library shrinks from ~127KB to ~83KB.
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