-
Notifications
You must be signed in to change notification settings - Fork 314
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Consolidate benchmark methods and give them a clearer naming system
Summary: * No longer have "default" like "SOBOL+BOTORCH_MODULAR::default"; instead, construct method from components (e.g. "SOBOL+BOTORCH_MODULAR::SingleTaskGP_qLogNoisyExpectedImprovement") and *test* that the benchmarks match defaults. * Remove non-Log EI benchmarks, since these have all been migrated. * Define MBM benchmarks more concisely and programmatically. Defining benchmark methods explicitly in terms of their components, without reference to defaults, has a few benefits: 1) One can't accidentally test the same method multiple times; 2) There is a 1:1 mapping between method name and functionality rather than many:many, making it easier to track the performance of methods, and 3) tests will prevent benchmarks from drifting apart from default behavior (in MBM or CGS) over time. Differential Revision: D49566866
- Loading branch information
1 parent
7a8069b
commit fe664ee
Showing
6 changed files
with
97 additions
and
337 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.