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

[AutoTVM] Add index boundary check in ConfigSpace.get() #7234

Merged
merged 2 commits into from
Jan 11, 2021

Conversation

ymwangg
Copy link
Contributor

@ymwangg ymwangg commented Jan 8, 2021

Currently autotvm.task.ConfigSpace.get() can take any number. This PR adds boundary check as well as fixing some typos.

@jcf94
Copy link
Contributor

jcf94 commented Jan 11, 2021

Thanks! @ymwangg @comaniac

@jcf94 jcf94 merged commit 69a0628 into apache:main Jan 11, 2021
tkonolige pushed a commit to tkonolige/incubator-tvm that referenced this pull request Jan 11, 2021
* [AutoTVM] Add index boundary check in ConfigSpace.get()

* Fix unit test

Co-authored-by: Yanming Wang <[email protected]>
masahi pushed a commit to masahi/tvm that referenced this pull request Jan 14, 2021
* [AutoTVM] Add index boundary check in ConfigSpace.get()

* Fix unit test

Co-authored-by: Yanming Wang <[email protected]>
masahi pushed a commit to masahi/tvm that referenced this pull request Jan 18, 2021
* [AutoTVM] Add index boundary check in ConfigSpace.get()

* Fix unit test

Co-authored-by: Yanming Wang <[email protected]>
TusharKanekiDey pushed a commit to TusharKanekiDey/tvm that referenced this pull request Jan 20, 2021
* [AutoTVM] Add index boundary check in ConfigSpace.get()

* Fix unit test

Co-authored-by: Yanming Wang <[email protected]>
trevor-m pushed a commit to neo-ai/tvm that referenced this pull request Jan 21, 2021
* [AutoTVM] Add index boundary check in ConfigSpace.get()

* Fix unit test

Co-authored-by: Yanming Wang <[email protected]>
electriclilies pushed a commit to electriclilies/tvm that referenced this pull request Feb 18, 2021
* [AutoTVM] Add index boundary check in ConfigSpace.get()

* Fix unit test

Co-authored-by: Yanming Wang <[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.

3 participants