镜像站点 · 本页由第三方 GitHub 只读镜像提供,非 GitHub 官方站点,不接受任何登录或凭据输入。前往 github.com
Skip to content

Make the ESP-IDF kernel bundle at build time when Python has executorch - #24707

Open
rascani wants to merge 1 commit into
esp-idf-kernel-bundlesfrom
esp-idf-auto-kernels
Open

rascani wants to merge 1 commit into
esp-idf-kernel-bundlesfrom
esp-idf-auto-kernels

Conversation

@rascani

@rascani rascani commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

When a project declares models but no kernel bundle, the component's configure
step now checks whether EXECUTORCH_PYTHON (set with -D or in the environment;
by default the Python ESP-IDF builds with) can import executorch. If it can, it
runs gen_kernels.py on the declared models into the build directory and uses
that bundle, so a project that exports and builds in one environment no longer
has to run gen_kernels.py or keep a bundle in its tree. The result is
byte-identical to a bundle made by hand, which CI checks by building the
ESP32-S3 smoke consumer this way. Without such a Python, or if generation
fails, the configure error names the Python, shows why, and prints the
gen_kernels.py command.

Authored with Claude Code.

@pytorch-bot

pytorch-bot Bot commented Nov 5, 2026 •

Copy link
Copy Markdown

🔗 Helpful Links

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

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

❌ 1 New Failure, 1 Unrelated Failure

As of commit 4c21d62 with merge base 60a8b5f (image):

NEW FAILURE - The following job has 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.

@rascani
rascani added this pull request to stack #24713 November 5, 2026 23:48
@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 Nov 5, 2026
@github-actions

github-actions Bot commented Nov 5, 2026

Copy link
Copy Markdown

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.

@rascani
rascani force-pushed the esp-idf-auto-kernels branch from 5e90ca5 to 5683d02 Compare November 6, 2026 19:19
@rascani
rascani force-pushed the esp-idf-auto-kernels branch from 5683d02 to 64553bc Compare November 6, 2026 20:40
@rascani
rascani requested a review from JakeStevens November 6, 2026 20:46
@rascani
rascani marked this pull request as ready for review November 6, 2026 20:46
@rascani
rascani requested a review from larryliu0820 as a code owner November 6, 2026 20:46
@rascani
rascani force-pushed the esp-idf-auto-kernels branch from 64553bc to 31ebbfa Compare November 6, 2026 21:15
When a project declares models but no kernel bundle, the component's configure
step now checks whether EXECUTORCH_PYTHON (set with -D or in the environment;
by default the Python ESP-IDF builds with) can import executorch. If it can, it
runs gen_kernels.py on the declared models into the build directory and uses
that bundle, so a project that exports and builds in one environment no longer
has to run gen_kernels.py or keep a bundle in its tree. The result is
byte-identical to a bundle made by hand, which CI checks by building the
ESP32-S3 smoke consumer this way. Without such a Python, or if generation
fails, the configure error names the Python, shows why, and prints the
gen_kernels.py command.

Authored with Claude Code.

This branch was previously deployed

1 inactive deployment
cadence — 4c21d62b Deployed Nov 7, 2026 by rascani via hifi-op-test / hifi4 #37200
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants