-
Couldn't load subscription status.
- Fork 702
Add Pico2 Tutorials on Raspberry Pi #15188
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
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/15188
Note: Links to docs will display an error until the docs builds have been completed. ❌ 2 New Failures, 20 PendingAs of commit c11c5b3 with merge base 1489d4f ( NEW FAILURES - The following jobs have failed:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
This PR needs a
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. I left a few small nits comments.
docs/source/pico2_tutorial.md
Outdated
| @@ -0,0 +1,199 @@ | |||
| # Pico2: A simple MNIST Tutorial | |||
|
|
|||
| Deploy your PyTorch models directly to Raspberry Pi Pico2 microcontroller with ExecuTorch and Infer. | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is Infer just referring to inference in this case, or is it some specific noun? Wording seems to imply that Infer is a separate tool or library.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed Infer
docs/source/pico2_tutorial.md
Outdated
| python export_mlp_mnist.py # Creates balanced_tiny_mlp_mnist.pte | ||
| ``` | ||
|
|
||
| - **Note:** This is hand-crafted MNIST Classifier (proof-of-concept), and not production trained.This tiny MLP recognizes digits 0, 1, 4, and 7 using manually designed feature detectors. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: Missing a space in "production trained.This..."?
docs/source/pico2_tutorial.md
Outdated
|
|
||
| - Train real models → Replace hand-crafted weights | ||
| - Optimize further → INT8 quantization, pruning | ||
| - Deploy fleet → Mass production ready |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: Reads slightly weird.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Worded it better
| #### Important Caveats: | ||
|
|
||
| - Memory constraints - Models must fit in 520KB SRAM | ||
| - Memory constraints - Models must fit in 520KB SRAMn (Pico2) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the n intended in SRAMn?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
|
@pytorchbot cherry-pick --onto release/1.0 -c docs |
### Summary Add Pico2 Tutorials on Raspberry Pi [PLEASE REMOVE] See [CONTRIBUTING.md's Pull Requests](https://github.com/pytorch/executorch/blob/main/CONTRIBUTING.md#pull-requests) for ExecuTorch PR guidelines. [PLEASE REMOVE] If this PR closes an issue, please add a `Fixes #<issue-id>` line. [PLEASE REMOVE] If this PR introduces a fix or feature that should be the upcoming release notes, please add a "Release notes: <area>" label. For a list of available release notes labels, check out [CONTRIBUTING.md's Pull Requests](https://github.com/pytorch/executorch/blob/main/CONTRIBUTING.md#pull-requests). ### Test plan [PLEASE REMOVE] How did you test this PR? Please write down any manual commands you used and note down tests that you have written if applicable.
Summary
Add Pico2 Tutorials on Raspberry Pi
[PLEASE REMOVE] See CONTRIBUTING.md's Pull Requests for ExecuTorch PR guidelines.
[PLEASE REMOVE] If this PR closes an issue, please add a
Fixes #<issue-id>line.[PLEASE REMOVE] If this PR introduces a fix or feature that should be the upcoming release notes, please add a "Release notes: " label. For a list of available release notes labels, check out CONTRIBUTING.md's Pull Requests.
Test plan
[PLEASE REMOVE] How did you test this PR? Please write down any manual commands you used and note down tests that you have written if applicable.