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

Auto-tuning a Convolutional Network for ARM CPU (tutorial error, bug reports) #8103

Merged
merged 3 commits into from
Jun 9, 2021

Conversation

cbswj
Copy link
Contributor

@cbswj cbswj commented May 21, 2021

issue link : #8067

It is a problem when using RPC session to connect Raspberry Pi and GPU server to proceed autoTVM.

There was a problem with tuning on autoTVM.

As in the tutorial example, creating xgbtuner causes problems.

I looked up tvm-API to solve the problem and found this sentence.
‘For cross-device or cross-operator tuning, you can use ‘curve’ only.’

However, the tutorial creates an xgbtuner that does not have a curve option.

In addition, the tutorial doesn't even have code to create xgbtuner with curve options.

Although it is an autoTVM tutorial for ARM CPU using RPC session, it seems strange that there is no curve option when creating xgbtuner.

So I made all the feature_type options available by xgbtuner and experimented.
Of the 3 options knob, itervar, and curve, only knob option does not fail and works well!

Question
The API says to use the curve option when using xgb, but in reality, it is strange that the knob option, not the curve option, works!

Copy link
Contributor

@leandron leandron left a comment

Choose a reason for hiding this comment

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

Hi @cbswj, thanks for the PR. I see you got some formatting issues. You can find some guidance on how to lint you patches here: https://tvm.apache.org/docs/contribute/pull_request.html#submit-a-pull-request

Regarding your question about the API and supported options for tuning, @jcf94 @comaniac can you help?

Copy link
Contributor

@comaniac comaniac left a comment

Choose a reason for hiding this comment

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

LGTM

@tqchen
Copy link
Member

tqchen commented Jun 4, 2021

gentle ping to resolve the lint error, you can run tests\lint\git-black.sh -i upstream/main to format the code

@jcf94
Copy link
Contributor

jcf94 commented Jun 7, 2021

@cbswj Hi, it nees you to fix the lint error following the https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-8103/2/pipeline before we can merge this.

I can help do that if the author didn't come back after several days.

@jcf94 jcf94 merged commit 9899f1e into apache:main Jun 9, 2021
@jcf94
Copy link
Contributor

jcf94 commented Jun 9, 2021

Now merged. Thanks @cbswj @leandron @comaniac @tqchen

trevor-m pushed a commit to trevor-m/tvm that referenced this pull request Jun 17, 2021
…reports) (apache#8103)

* tune_relay_arm.py tutorial modify

* Lint fix

* Re-trigger CI

Co-authored-by: Chenfan <[email protected]>
trevor-m pushed a commit to neo-ai/tvm that referenced this pull request Jun 17, 2021
…reports) (apache#8103)

* tune_relay_arm.py tutorial modify

* Lint fix

* Re-trigger CI

Co-authored-by: Chenfan <[email protected]>
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.

5 participants