-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Add logging with state to Bayesian Optimizer #547
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
till-m
merged 31 commits into
bayesian-optimization:master
from
adrianmolzon:overhaul-logging
Mar 17, 2025
Merged
Changes from 21 commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
58f48d3
Add functionality to save and load state of the BayesianOptimization
adrianmolzon 71036c6
Update basic-tour with new save and load functionality
adrianmolzon 84d036d
move load stateful path to optional argument in class instantiation
adrianmolzon 6ae61d9
add test for string params, update tests with new load functionality
adrianmolzon 7be3854
updated basic tour with updated paths
adrianmolzon 2b514aa
add the random state to the set of things to list of saved items
adrianmolzon be73262
move state loading to separate function, add functionality for saving…
adrianmolzon ad2c822
use new loading schema
adrianmolzon aa86e2f
update tests, add integration tests for saving and loading acquisitio…
adrianmolzon c26504e
undo abstractmethod implementation for get and set state saving funct…
adrianmolzon 4aab0b8
reorganize state saving and loading for consistency
adrianmolzon 14ea11a
move integration tests into acquisition
adrianmolzon a782960
remove unndecessary test, add tests for domain reduction and custom p…
adrianmolzon b209c64
make test more comprehensive
adrianmolzon 79b701c
add test logs
adrianmolzon 7d6b9d6
sync execution counts from basic tour
adrianmolzon ab765b4
linting, whitespace removal, import structuring
adrianmolzon c2ea551
ruff fix for string literal in error message
adrianmolzon c21c6c6
fix ruff complaints
adrianmolzon 57092d9
make all side param comparisons almost equal to account for slight nu…
adrianmolzon 9e57bff
reformat array comparison check
adrianmolzon 289a0d5
upgrade poetry2.0 & apply pep621 (#545)
phi-friday d0ef58a
Fix coverage report (#552)
till-m a68d727
remove unnecessary files, have acquisition baseclass functions raise …
adrianmolzon 3d3e538
remove duplicate acquisition functions random state
adrianmolzon cf87c7b
ruff format
adrianmolzon b5ae882
add type hints for base acquisition get/set functions
adrianmolzon 02d2643
remove noreturn
adrianmolzon 2f8ab64
remove former saving functionality from notebooks
adrianmolzon 413f467
increase legibility of custom acquisition example
adrianmolzon 472fd93
explicitly stating the optionality of the saving and loading in custo…
adrianmolzon File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.