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.

Sign up for GitHub

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

Jump to bottom

[TOPI] Setting up AutoTVM template for Intel Int8 conv2D #3955

Merged
merged 1 commit into from
Sep 16, 2019
Merged

[TOPI] Setting up AutoTVM template for Intel Int8 conv2D #3955

merged 1 commit into from
Sep 16, 2019

Conversation

anijain2305
Copy link
Contributor

@comaniac
Copy link
Contributor

Would you consider the fallback config?

@anijain2305 anijain2305 force-pushed the autotvm branch 2 times, most recently from 7deb41d to 23a8107 Compare September 15, 2019 07:22
@anijain2305
Copy link
Contributor Author

Would you consider the fallback config?

@comaniac , the fallback config is already set. Here, we directly use the NCHW fallback config. This happens during the alter_op_layout conversion from conv2d to conv2d_nchwc_int8.

Next week, I am going to a do a much larger sweep of ic and oc to check robustness. I can incrementally build then.

@comaniac
Copy link
Contributor

@comaniac , the fallback config is already set. Here, we directly use the NCHW fallback config. This happens during the alter_op_layout conversion from conv2d to conv2d_nchwc_int8.
Next week, I am going to a do a much larger sweep of ic and oc to check robustness. I can incrementally build then.

Make sense. Thanks for the clarification.

kevinthesun
kevinthesun approved these changes Sep 16, 2019
View reviewed changes
Copy link
Contributor

@kevinthesun kevinthesun left a comment

Choose a reason for hiding this comment

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

LGTM

@yzhliu yzhliu merged commit 3edf526 into apache:master Sep 16, 2019
@yzhliu
Copy link
Member

yzhliu commented Sep 16, 2019

Thanks @anijain2305 @comaniac @kevinthesun

lxhsjtu added a commit to lxhsjtu/tvm that referenced this pull request Sep 17, 2019
@lxhsjtu
Revert "[TOPI] Setting up AutoTVM template for Intel Int8 conv2D (apa…
6099eb3 
…che#3955)"

This reverts commit 3edf526.
wweic pushed a commit to wweic/tvm that referenced this pull request Sep 30, 2019
@anijain2305 @wweic
[TOPI] Setting up AutoTVM template for Intel Int8 conv2D (apache#3955)
efac88b
wweic pushed a commit to wweic/tvm that referenced this pull request Sep 30, 2019
@anijain2305 @wweic
[TOPI] Setting up AutoTVM template for Intel Int8 conv2D (apache#3955)
25ce696
wweic pushed a commit to neo-ai/tvm that referenced this pull request Oct 1, 2019
@anijain2305 @wweic
[TOPI] Setting up AutoTVM template for Intel Int8 conv2D (apache#3955)
492e27a
@tqchen tqchen mentioned this pull request Nov 8, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kevinthesun kevinthesun kevinthesun approved these changes

@yzhliu yzhliu yzhliu 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.
4 participants
@anijain2305 @comaniac @yzhliu @kevinthesun