Skip to content
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

Allow use of SDV to bias optimizer samples. #1

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

tlranda
Copy link

@tlranda tlranda commented Jun 21, 2022

Files changed in this commit are referenced from:
https://github.com/deephyper/deephyper/tree/master/deephyper/skopt

An additional change is made to space/space.py to prevent a known Exception when probabilities are not explicitly defined by Categorical hyperparameters.

Files changed in this commit are referenced from:
https://github.com/deephyper/deephyper/tree/master/deephyper/skopt

An additional change is made to space/space.py to prevent Exception when
probabilities are not explicitly defined by Categorical hyperparameters.
@jaehoonkoo jaehoonkoo self-requested a review June 22, 2022 18:30
Copy link
Collaborator

@jaehoonkoo jaehoonkoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I found that CCS part is removed in the new script. We should keep CCS part.

@tlranda
Copy link
Author

tlranda commented Jun 22, 2022

Restored CCS in space.py based on the diff for ytopt-team's master branch for scikit-optimize

Prevent skopt optimizers from selecting rows they already have selected
Attempt to prevent unconstrained models from returning zero rows when generation fails to be 'close'
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants