Skip to content

[microTVM] Check the output of microNPU demos in CI#15667

Merged
lhutton1 merged 2 commits into
apache:mainfrom
lhutton1:check-demo
Sep 6, 2023
Merged

[microTVM] Check the output of microNPU demos in CI#15667
lhutton1 merged 2 commits into
apache:mainfrom
lhutton1:check-demo

Conversation

@lhutton1

@lhutton1 lhutton1 commented Sep 4, 2023

Copy link
Copy Markdown
Contributor

This is a followup from #15649 which caught a misclassification issue when running the microNPU demo. While #15649 fixes the issue and results in the correct classification, this commit ensures the demo does not regress again in the future.

cc @ashutosh-arm @leandron

This is a followup from apache#15649 which caught a misclassification
issue when running the microNPU demo. While apache#15649 fixes the issue
and results in the correct classification, this commit ensures the
demo does not regress again in the future.

Change-Id: I816bb7dbba92cf447ff66f26ff881e1586740b6a

@asparkhi asparkhi 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 @lhutton1 for catching this. I have added few queries.

Comment thread tests/scripts/task_demo_microtvm.sh Outdated
Comment thread tests/scripts/task_demo_microtvm.sh Outdated
Change-Id: I2cb25dce1e76890e3c72707146cccf0e43dc5456

@asparkhi asparkhi 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.

LGTM. Appreciate the effort. Thanks!

@lhutton1 lhutton1 merged commit 5d3f3dd into apache:main Sep 6, 2023
@lhutton1 lhutton1 deleted the check-demo branch September 6, 2023 14:09
@lhutton1

lhutton1 commented Sep 6, 2023

Copy link
Copy Markdown
Contributor Author

Thanks @ashutosh-arm!

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.

2 participants