Skip to content

Default Export for Mv2 with Real Image Input #13019

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Aug 7, 2025

Conversation

BujSet
Copy link
Contributor

@BujSet BujSet commented Jul 30, 2025

Summary

When testing executorch inference runners on real MCUs, it's helpful if the default export of the model includes a real input (rather than just filled with random values or all 1.0s). This PR does this for the mobilenet_v2 model, following the steps from the tutorial here.

Test plan

Ran python -m examples.portable.scripts.export --model_name="mv2" and saw the PTE now contains values associated with the dog.jpg image.

Copy link

pytorch-bot bot commented Jul 30, 2025

🔗 Helpful Links

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

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

❌ 2 New Failures, 1 Pending, 1 Unrelated Failure

As of commit 7883a76 with merge base 5797608 (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.

@BujSet BujSet self-assigned this Jul 30, 2025
@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 Jul 30, 2025
@BujSet BujSet requested a review from larryliu0820 July 30, 2025 22:39
@BujSet
Copy link
Contributor Author

BujSet commented Jul 30, 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 30, 2025
@BujSet BujSet marked this pull request as ready for review July 30, 2025 23:00
@BujSet BujSet requested review from lucylq and jackzhxng as code owners July 30, 2025 23:00
@BujSet BujSet force-pushed the mv2_with_real_input_clean branch from dae9395 to 7883a76 Compare August 7, 2025 17:50
@BujSet BujSet merged commit c52f6a0 into pytorch:main Aug 7, 2025
99 of 102 checks passed
@BujSet BujSet deleted the mv2_with_real_input_clean branch August 7, 2025 19:22
BujSet added a commit that referenced this pull request Aug 8, 2025
### Summary
[PR](#13019) forgot to change
the return example input to use the default samoyed dog image when using
real inputs. This PR fixes that typo.

### Test plan
Now the example runners that use the `mv2` export contain the dog image
in the pte by default.
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. 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