-
Notifications
You must be signed in to change notification settings - Fork 43
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Batch trust regions implementation of TURBO (#791)
* Add support for local models and datasets (WIP) * Add unit test for local models (WIP) * Update multi model/dataset test (WIP) * Add unit test for keep datasets in regions * Add more tests and move to local tags class * Always include global dataset in mapping * Add filter_mask method to trust region * Add more testing * Fix mypy model type issues * Add ask_tell testing * Fix summary init when only global dataset * Remove walrus operator * Update test, ask_tell data not changed in-place * Add some test comments * Add some rule comments * Allow input-multi-observers for batch observer * Allow multiple models/datasets for base rule * Support multiple models/datasets in region selects * Fix TR plotting history colors * Add notebook init points explanation * Rename region index and add init param * WIP * Add more TURBO implementation + attempt type fixes * Remove old comment * Tidy-up redundant expression * Fix TURBOBox and temp changes in TURBO for match * Add intermediate box region class * Keep full datasets along with filtered ones * Move subspace update to a new rule method * Add temp notebook for TURBO comparisons * Save TR subspaces in acquire to re-use later * Update notebook to use TURBOBOx * Make changes from PR feedback * Fix rename after merge * Fix compare notebook after merge * Move rule create later and tidy filtering dataset * More testing in notebook * Remove redundant dataset filtering * Address some of the recent feedback * Fix dataset mypy error * Copy dataset in optimizers to avoid changing it * Share DatasetChecker and tidy-up exp values in tests * Address more feedback * Remove prev TURBO and update tests to use new class * Remove notebook for testing * Create dataset and update at start of optim * Avoid default num_models in integ tests * Fix old python typing issue * Address feedback * Address more comments * Only copy state with track_state==True * Add comment explaining copy * Deepcopy subspace internal to rule * Keep global datasets unfiltered * Add notebook intro and improve TREGO text * Use flatten_... func and add comment * Improve TR explanations * Fix merge issues * Clarify parallel acq comments
- Loading branch information
1 parent
f766953
commit cf09b0f
Showing
12 changed files
with
806 additions
and
642 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
Oops, something went wrong.