From 9b4428c668bdcfbe282d689f7e703e8e9eace76e Mon Sep 17 00:00:00 2001 From: goulou Date: Tue, 23 Jun 2020 23:43:27 +0200 Subject: [PATCH] [ASHA][docs] Change default value of "brackets" (#9101) Closes #9058 --- doc/source/tune/api_docs/schedulers.rst | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/doc/source/tune/api_docs/schedulers.rst b/doc/source/tune/api_docs/schedulers.rst index 4d03e276e..f530aa4f0 100644 --- a/doc/source/tune/api_docs/schedulers.rst +++ b/doc/source/tune/api_docs/schedulers.rst @@ -66,11 +66,13 @@ The `ASHA `__ scheduler can be used b max_t=100, grace_period=10, reduction_factor=3, - brackets=3) + brackets=1) tune.run( ... , scheduler=asha_scheduler) Compared to the original version of HyperBand, this implementation provides better parallelism and avoids straggler issues during eliminations. **We recommend using this over the standard HyperBand scheduler.** An example of this can be `found here `_. +Even though the original paper mentions a bracket count of 3, discussions with the authors concluded that the value should be left to 1 bracket. This is the default used if no value is provided for the ``brackets`` argument. + .. autoclass:: ray.tune.schedulers.AsyncHyperBandScheduler .. autoclass:: ray.tune.schedulers.ASHAScheduler