Skip to content

[microTVM] AOT Demo - #8075

Merged
areusch merged 20 commits into
apache:mainfrom
mehrdadh:aot_zephyr
May 28, 2021
Merged

[microTVM] AOT Demo#8075
areusch merged 20 commits into
apache:mainfrom
mehrdadh:aot_zephyr

Conversation

@mehrdadh

@mehrdadhmehrdadh commented May 19, 2021•
edited
Loading

Copy link
Copy Markdown
Member

This PR makes these changes:

  • adds AOT demo for microtvm
  • adds AOT zephyr test to ci_qemu
  • adds image classification tflite model for aot zephyr test.

@mehrdadh

mehrdadh commented May 19, 2021

Copy link
Copy Markdown
MemberAuthor

cc @areusch@gromero

@mehrdadh
mehrdadhforce-pushed the aot_zephyr branch 2 times, most recently from 15b61d2 to a397a6bCompareMay 19, 2021 06:23
areusch
areusch requested changes May 20, 2021
View reviewed changes

@areuschareusch left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

just one minor question so far

Comment threadapps/microtvm/zephyr/demo_runtime/boards/nrf5340dk_nrf5340_cpuapp.conf
Comment threadapps/microtvm/zephyr/demo_runtime/CMakeLists.txt Outdated
@mehrdadh

mehrdadh commented May 21, 2021

Copy link
Copy Markdown
MemberAuthor

@areusch tested this on NRF board, however, the model doesn't fit on Nucleo board. So I removed the boards that has not been tested so far in the aot_demo.
PTAL.
Thanks!

areusch
areusch requested changes May 24, 2021
View reviewed changes

@areuschareusch left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

thanks @mehrdadh , did a pass

Comment threadapps/microtvm/zephyr/aot_demo/boards/qemu_x86.conf
Comment threadapps/microtvm/zephyr/aot_demo/include/zephyr_uart.h
Comment threadapps/microtvm/zephyr/aot_demo/src/main.c
Comment threadapps/microtvm/zephyr/host_driven_demo/CMakeLists.txt
Comment threadpython/tvm/micro/build.py Outdated
Comment threadtests/micro/zephyr/test_zephyr_aot.py Outdated
lowered = relay.build(relay_mod, target, params=params)

# Load sample and generate input/output header files
sample = np.load(os.path.join(this_dir, "testdata", "ic_sample_fp32_8.npy"))

@areuschareuschMay 24, 2021

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

311KB is quite a bit for this repo...i'm not sure whether we should load from a different place?

@mehrdadhmehrdadhMay 24, 2021

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

replaced it with this link to its source:
image classification model

@areuschareuschMay 25, 2021

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

thanks, also possible to do so for this one?

@mehrdadhmehrdadhMay 25, 2021

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

For the sample data I need to use a sample that has predictable result to test in the CI. I got this sample from a large dataset and need to keep it somewhere. The other option is to create a personal repository and add it there. We could reuse that repo later for other other samples as well. Let me know what you think.
Also I can save this numpy array as int8 and reduce the size 4 times if that works.

@areuschareusch mentioned this pull request May 25, 2021
Merged
areusch
areusch approved these changes May 28, 2021
View reviewed changes

@areuschareusch left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

thanks @mehrdadh !

@areusch
areusch merged commit dad59be into apache:mainMay 28, 2021
@mehrdadh
mehrdadh deleted the aot_zephyr branch June 10, 2021 19:51
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request Jun 17, 2021
@mehrdadhTrevor Morris
[microTVM] AOT Demo (apache#8075)
6b378f8
* initial
* remove compare
* temp fix
* debugging
* hack
* hack for testing
* both test pass
* cleanup
* fix tests and tutorials
* restructure
* cleanup
* cleanup
* fix check files
* fixed for physical devices
* address comments
* reduce nrf stack size
* update sample url
* format
trevor-m pushed a commit to neo-ai/tvm that referenced this pull request Jun 17, 2021
@mehrdadh@trevor-m
[microTVM] AOT Demo (apache#8075)
a3cab8d
* initial
* remove compare
* temp fix
* debugging
* hack
* hack for testing
* both test pass
* cleanup
* fix tests and tutorials
* restructure
* cleanup
* cleanup
* fix check files
* fixed for physical devices
* address comments
* reduce nrf stack size
* update sample url
* format
@junrushaojunrushao mentioned this pull request Nov 1, 2021
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@areuschareuschareusch approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mehrdadh@areusch