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

Bayesian optimization tutorial #416

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

Conversation

hurricane642
Copy link

In this PR I add an example of using Bayesian optimization for Power Spectrum. Most of the code is based on the first librascal example, I have not changed the meaningful part of it, adding only a few things about the optimization itself. This is a fairly simple example with the simplest functions from the scikit-optimize library, but I hope it is enough for the first acquaintance. This is my first time writing examples for the library, so any suggestions and edits are welcome.

@max-veit
Copy link
Contributor

max-veit commented Aug 4, 2022

Very good, thanks for adding this! I probably won't have time for a full review anytime soon (maybe late next week but no promises). But right now my main comment is that there's a lot of utility code, especially in the second-to-last cell, that would benefit from either being moved to its own "notebook utilities" module or broken down and explained step-by-step in the notebook.

BTW the tests are failing because of an update in scikit-cosmo a while back; there's a fix in 95aedf9

@hurricane642
Copy link
Author

Very good, thanks for adding this! I probably won't have time for a full review anytime soon (maybe late next week but no promises). But right now my main comment is that there's a lot of utility code, especially in the second-to-last cell, that would benefit from either being moved to its own "notebook utilities" module or broken down and explained step-by-step in the notebook.

BTW the tests are failing because of an update in scikit-cosmo a while back; there's a fix in 95aedf9

Thank you so much for your reply! I've tried to account for the errors, added more comments on how the code works and broken it down into separate parts. I wanted to clarify what you meant about the test. That is, how do I make the tests pass?:)

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