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

286 add surrogate models that can extrapolate #300

Merged
merged 26 commits into from
Oct 20, 2023

Conversation

dlinzner-bcs
Copy link
Contributor

  • again the stupid bug that the surrogate has to be allowed for BotochSurrogate

@dlinzner-bcs
Copy link
Contributor Author

@bertiqwerty the test is there :) can you please check?

Copy link
Contributor

@bertiqwerty bertiqwerty left a comment

Choose a reason for hiding this comment

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

As discussed.

@dlinzner-bcs dlinzner-bcs merged commit 7938ac5 into main Oct 20, 2023
dlinzner-bcs added a commit that referenced this pull request Oct 20, 2023
* add polynomial kernel

* add polynomial kernel

* add polynomial kernel

* add polynomial kernel

* add quadratic surrogate

* adding linear kernel not necessary

* adding linear kernel not necessary

* adding linear kernel not necessary

* fix test

* change kernel tzpe hint

* generalize quadratic surrogate

* generalize quadratic surrogate

* generalize quadratic surrogate

* fix init

* fix init

* fix init

* fix init

* fix init

* add polynomial_surrogate to AnyBotochSurrogate

* add test that botorch surrogate can be composed

* add test that botorch surrogate can be composed

* add test that botorch surrogate can be composed

* add test that botorch surrogate can be composed

* add test that botorch surrogate can be composed
dlinzner-bcs added a commit that referenced this pull request Oct 23, 2023
* add polynomial kernel

* add polynomial kernel

* add polynomial kernel

* add polynomial kernel

* add quadratic surrogate

* adding linear kernel not necessary

* adding linear kernel not necessary

* adding linear kernel not necessary

* fix test

* change kernel tzpe hint

* generalize quadratic surrogate

* generalize quadratic surrogate

* generalize quadratic surrogate

* fix init

* fix init

* fix init

* fix init

* fix init

* add polynomial_surrogate to AnyBotochSurrogate

* add test that botorch surrogate can be composed

* add test that botorch surrogate can be composed

* add test that botorch surrogate can be composed

* add test that botorch surrogate can be composed

* add test that botorch surrogate can be composed

* added test for iterative strategy

* added iterative strategy

* 286 add surrogate models that can extrapolate (#300) (#303)

* add polynomial kernel

* add polynomial kernel

* add polynomial kernel

* add polynomial kernel

* add quadratic surrogate

* adding linear kernel not necessary

* adding linear kernel not necessary

* adding linear kernel not necessary

* fix test

* change kernel tzpe hint

* generalize quadratic surrogate

* generalize quadratic surrogate

* generalize quadratic surrogate

* fix init

* fix init

* fix init

* fix init

* fix init

* add polynomial_surrogate to AnyBotochSurrogate

* add test that botorch surrogate can be composed

* add test that botorch surrogate can be composed

* add test that botorch surrogate can be composed

* add test that botorch surrogate can be composed

* add test that botorch surrogate can be composed

* rmv unneccessary private flagged  variables

* made strategy selection clearer
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