-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Improve the x86 auto-tune tutorial #3609
Conversation
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.
how does ci fail though ...
I rebased the PR, but it still can't pass CI. After reading the log, I don't think this PR has anything to do with that build error. |
No space left on device |
@tqchen Could you look into that? |
The space issue should have been fixed |
This CI build passes this time. |
Thanks! |
Thanks for contributing to TVM! Please refer to guideline https://docs.tvm.ai/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers.
Previously,
data
was directly used as an input name for the graph. For new users, they may be confused sincedata
also sounds like a constant property name. This PR tries to extract it out and make a variable so that it will be clear for the user if they want to auto-tune their own graph.