-
Notifications
You must be signed in to change notification settings - Fork 44
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
Add Bayesian Optimization into HiOp #711
Conversation
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.
see the comments. I think we should merge this without updating the user manual.
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 think this is a great first time and don't have any big concerns that prevent me from approving this but do have a few questions. Are the smtProblems omitted in order to reduce the amount of code in hiopbbpy? Should we change the name from hiopbbpy to hioppy? While hiopbbpy is intended for use in black box BO optimization. If we eventually make hiop algorithms available in the hiopbbpy interface then it seems that a user could just use of the hiop algorithms in any python application.
This PR adds a GP-based Bayesian Optimization (BO) solver into HiOp.