Skip to content

[tune] Update ZOOpt to better support the latest Ray#11462

Merged
richardliaw merged 5 commits intoray-project:masterfrom
zewenli98:master
Oct 19, 2020
Merged

[tune] Update ZOOpt to better support the latest Ray#11462
richardliaw merged 5 commits intoray-project:masterfrom
zewenli98:master

Conversation

@zewenli98
Copy link
Copy Markdown
Contributor

Why are these changes needed?

Add a parallel_num argument that allows users to specify the num of workers to parallel in zoopt.
ZOOpt package supports grid type as of v0.4.1.

This PR fixes some known bugs and supports the latest Ray and zoopt.

Checks

  • I've run scripts/format.sh to lint the changes in this PR.
  • I've included any doc changes needed for https://docs.ray.io/en/master/.
  • I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/
  • Testing Strategy
    • Unit tests
    • Release tests
    • This PR is not tested :(

@zewenli98 zewenli98 changed the title Update ZOOpt to better support the latest Ray [tune] Update ZOOpt to better support the latest Ray Oct 19, 2020
@richardliaw richardliaw merged commit 202b185 into ray-project:master Oct 19, 2020
zewenli98 added a commit to zewenli98/ray that referenced this pull request Oct 20, 2020
@zewenli98
Copy link
Copy Markdown
Contributor Author

I apologize that there is a bug in doc forgotten to fix, and I will raise a new PR.

@zewenli98 zewenli98 mentioned this pull request Oct 20, 2020
6 tasks
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.

2 participants