Skip to content
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

[microTVM] Project API based Arduino support #8493

Closed
wants to merge 47 commits into from

Conversation

guberti
Copy link
Member

@guberti guberti commented Jul 17, 2021

This PR adds support for generating Arduino projects to microTVM. Using an arbitrary model as input, we create a bare-bones Arduino project that can instantiate and run inference on the model. The developer is then free to build their Arduino project using this functionality.

Host-driven inference is not currently supported, but a future PR will add this functionality. Arduino support in this PR implements Project API, so this PR depends on #8380 and will merge afterwards.

@areusch @mdw-octoml

areusch and others added 29 commits July 23, 2021 13:29
 * use template crt_config.h for host test runtime; delete
   src/runtime/crt/host/crt_config.h so that it doesn't diverge from
   the template
 * bring template crt_config.h inline with the one actually in use
  * rename to MAX_STRLEN_DLTYPE
 * Create a dedicated TVM-side host crt_config.h in src/runtime/micro
 * move all zephyr projects to apps/microtvm/zephyr/template_project
Copy link
Contributor

@areusch areusch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks @guberti, this is really exciting! i left some early feedback on the PR while we wait for Project API server to merge.

@guberti
Copy link
Member Author

guberti commented Jul 27, 2021

Thanks for the early feedback @areusch! Your suggestions all made sense, and should be fixed with commits 4baf6d5, 5453d02, 6c434b9, and 1516f18.

@mehrdadh
Copy link
Member

mehrdadh commented Aug 9, 2021

@guberti I think you need to rebase this.

@guberti
Copy link
Member Author

guberti commented Aug 10, 2021

Replaced by #8708.

@guberti guberti closed this Aug 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants