[tune] with-params fix #12522

richardliaw · 2020-12-01T04:33:34Z

Why are these changes needed?

Properly moves data through object store (adds tests too)

closes #12521

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 :(

Signed-off-by: Richard Liaw <rliaw@berkeley.edu>

krfricke

Thanks for spotting and solving this.

Signed-off-by: Richard Liaw <rliaw@berkeley.edu>

richardliaw added 2 commits November 30, 2020 20:22

fix-with-params

636a2d7

Signed-off-by: Richard Liaw <rliaw@berkeley.edu>

fix

c149a8a

Signed-off-by: Richard Liaw <rliaw@berkeley.edu>

richardliaw assigned krfricke Dec 1, 2020

Merge branch 'master' into fix-with-params

a857fff

krfricke approved these changes Dec 1, 2020

View reviewed changes

richardliaw added 2 commits December 1, 2020 12:57

Merge branch 'master' into fix-with-params

fc815b4

lint

44c3fe7

Signed-off-by: Richard Liaw <rliaw@berkeley.edu>

richardliaw merged commit 4dc1673 into ray-project:master Dec 2, 2020

richardliaw deleted the fix-with-params branch December 2, 2020 00:47