-
Notifications
You must be signed in to change notification settings - Fork 11
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
Added hypervolume acquisition and constraints #45
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Specify latent dimensions
CI: install packages via built-in system Python (not conda)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I skimmed through the PR, looks good!
We need to unpin bluesky to be able to deploy on the floor with bluesky v1.10.0.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we unpin bluesky
as well, but add zict<3.0.0
like in https://github.com/bluesky/bluesky/blob/6e3c4721ce728bcea2d2cc64f3e56d7c6b31e5e5/requirements.txt#L12C1-L12C11?
Added hypervolume acquisition and constraints
Optimization constraints in botorch are weird, so we should just implement our own acquisition functions that we can manually adjust using the output of the classifier.