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

[Doc][AutoTVM] Fix bugs that override n_trials #4842

Merged
merged 1 commit into from
Feb 7, 2020

Conversation

comaniac
Copy link
Contributor

@comaniac comaniac commented Feb 7, 2020

In the loop of tuning a list of tasks, n_trials will be overwritten and result in incorrect maximum trial number for rest tasks.

cc @dati91 @merrymercy @eqy

@comaniac
Copy link
Contributor Author

comaniac commented Feb 7, 2020

Also cc @zxy844288792 @mole10.

@comaniac comaniac changed the title [Doc][AutoTVM] Fix bugs that overrides n_trials [Doc][AutoTVM] Fix bugs that override n_trials Feb 7, 2020
@vinx13 vinx13 merged commit e33de24 into apache:master Feb 7, 2020
@comaniac comaniac deleted the fix_tutorial_bug branch February 7, 2020 22:57
anijain2305 pushed a commit to anijain2305/tvm that referenced this pull request Feb 10, 2020
alexwong pushed a commit to alexwong/tvm that referenced this pull request Feb 26, 2020
alexwong pushed a commit to alexwong/tvm that referenced this pull request Feb 28, 2020
zhiics pushed a commit to neo-ai/tvm that referenced this pull request Mar 2, 2020
tqchen pushed a commit to tqchen/tvm that referenced this pull request Mar 29, 2020
@@ -255,7 +255,7 @@ def tune_tasks(tasks,
os.remove(tmp_log_file)

for i, tsk in enumerate(reversed(tasks)):
prefix = "[Task %2d/%2d] " % (i+1, len(tasks))
Copy link

Choose a reason for hiding this comment

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

能询问一下这句话的意思么?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants