Skip to content

Commit

Permalink
Change default acquisition function back to MES
Browse files Browse the repository at this point in the history
PVRS turned out to be too explorative in high dimensions which makes it
an unsuitable choice as default acquisition function.
PVRS is still a very robust choice for 3 or less parameters.
  • Loading branch information
kiudee committed Oct 13, 2020
1 parent 2da2a6a commit b80cdf3
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions docs/parameters.myst
Original file line number Diff line number Diff line change
Expand Up @@ -142,10 +142,10 @@ fitting process:
* - `"acq_function"`
- `-a --acq-function TEXT`
- The search strategy used by the tuner. The following values are possible:
* `"mes"`: Max-value entropy search tries to sample points which
* `"mes"` (default): Max-value entropy search tries to sample points which
yield a lot of information about the value of the optimum.
A good allrounder strategy leaning on the exploitative side.
* `"pvrs"` (default): Predictive variance reduction search tries to sample
* `"pvrs"`: Predictive variance reduction search tries to sample
points which reduce the uncertainty of promising regions as much as
possible. Another good allrounder strategy, which compared to `mes`
focuses more on exploration. Is slightly more robust.
Expand Down
4 changes: 2 additions & 2 deletions tune/cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ def run_server(verbose, logfile, command, experiment_file, dbconfig):
@click.option(
"-a",
"--acq-function",
default="pvrs",
default="mes",
help="Acquisition function to use for selecting points to try. "
"Can be {mes, pvrs, ei, ts, vr}.",
show_default=True,
Expand Down Expand Up @@ -236,7 +236,7 @@ def run_server(verbose, logfile, command, experiment_file, dbconfig):
)
def local( # noqa: C901
tuning_config,
acq_function="pvrs",
acq_function="mes",
acq_function_samples=1,
confidence=0.9,
data_path=None,
Expand Down

0 comments on commit b80cdf3

Please sign in to comment.