-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[microTVM][tvmc] Add TVMC Micro tutorial for Zephyr #10024
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
ae7b1e9 to
670a20a
Compare
|
cc @hogepodge |
gromero
left a comment
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.
@mehrdadh nice job! Thanks for adding that tutorial. Just some small fixes (see comments inline) and I think it's pretty much ready to land.
The other fixes in tvmc tests autotuning tutorial are good.
|
@gromero thanks for the review! I added your changes, PTAL. |
| # Run Tiny Model on Micro Target | ||
| ############################################################ | ||
| # | ||
| # To run the flashed model on the device using TVMC, we use ``tvmc run`` subcommand and we pass ``--device micro`` |
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.
could you explain what exactly has happened now at this point, and what we mean by "run the flashed model on the device?"
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.
@areusch added more details. Please let me know if I should mention anything else. thanks!
| # zephyr \ | ||
| # --project-option project_type=host_driven zephyr_board=qemu_x86 | ||
| # | ||
| # This will generate a Zephyr project for `qemu_x86` zephyr board. To get more information about |
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.
could you explain host_driven here?
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.
Added some details, PTAL. thanks!
gromero
left a comment
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.
@mehrdadh Thanks for the v2. LGTM.
|
Thanks for this, read through it and it looks good to me. |
areusch
left a comment
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.
Thanks @mehrdadh @hogepodge @driazati @gromero !
This PR adds a TVMC tutorial for Zephyr platforms using
tvmc micro.