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

WIP: Batch vs single #150

Merged
merged 10 commits into from
Aug 30, 2023
Merged

WIP: Batch vs single #150

merged 10 commits into from
Aug 30, 2023

Conversation

SRFU-NN
Copy link
Collaborator

@SRFU-NN SRFU-NN commented Mar 22, 2023

Showcasing how to use batches of experiments

@@ -17,8 +17,8 @@ Can GitHub Actions run the notebooks and check for errors? (Treebeard?)
3. "Good enough" found during initialisation
4. batch vs. single opt.ask()
5. Noise
6. Narrow wells
7. Explore vs. exploit (Gold Diggerson vs. Map Drawson)
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think explore vs. exploit is the natural next place to go, since we have just talked about strategies.

@@ -73,7 +73,7 @@
],
"source": [
"def score(coordinates, verbose = True):\n",
" \"\"\" Modified Branin-Hoo function. It has three minima with f(x*) = 0.397887 \n",
" \"\"\" Modified Branin-Hoo function. It has three local minima.\n",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I have simplified the score function from the standard Branin-Hoo, so it doesn't have as nice analytical properties anymore. We don't need the optima to have exactly identical scores, so I value the simplicity of the function higher.

@SRFU-NN SRFU-NN requested a review from sqbl March 23, 2023 10:02
@SRFU-NN SRFU-NN added the documentation Improvements or additions to documentation label Mar 27, 2023
@sqbl
Copy link
Collaborator

sqbl commented Mar 29, 2023

My Nitty-gritties:

  • replay cells from top
  • In the coding box "Getting several suggestions at once", fifth line says five new experiments
  • The chapter "Knowledge Gap" is not updated with the new batch size
  • Typo is -> it (chapter "Strategies")
  • Typo ProcessOptmizer -> ProcessOptimizer (chapter "Strategies")
  • Typo aat -> at (chapter "Strategies")
  • Typo uch -> much (in last codeblock line 12

Could you add a concluding remark?

On a general note, I like this story line and the "justifications" for showing the code and its capabilities.

@SRFU-NN
Copy link
Collaborator Author

SRFU-NN commented Mar 29, 2023 via email

@SRFU-NN SRFU-NN changed the title Batch vs single WIP: Batch vs single Apr 11, 2023
@SRFU-NN SRFU-NN merged commit 48c5128 into new_examples Aug 30, 2023
1 check passed
@SRFU-NN SRFU-NN deleted the batch_vs_single branch August 30, 2023 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants