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

Add a tutorial for the new Warm-start feature #1029

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

lebrice
Copy link
Collaborator

@lebrice lebrice commented Nov 22, 2022

Adds an example of how to use the new warm-start feature.

Currently uses the Profet tasks, which require quite a bit of compute to run the first time (GPUS also help, but aren't required).

TODOS:

  • Use more lightweight, realistic tasks.
  • Make it pretty.

Current results:
warm_start_example

@lebrice lebrice force-pushed the warm_start_example branch 2 times, most recently from ec69241 to cec5d5a Compare November 22, 2022 22:43
@lebrice lebrice marked this pull request as ready for review December 6, 2022 19:13
storage=current_experiment_storage,
executor=SingleExecutor(),
max_trials=current_experiment_n_runs,
# NOTE: This n_initial_points is changed slightly, since it also counts the trials from
Copy link
Member

Choose a reason for hiding this comment

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

It's interesting to make this fair comparison. In practice however would not need to force random points first at the previous results already provide a good basis to start bayesian optimization.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ok, should I keep this then? or not?

Signed-off-by: Fabrice Normandin <[email protected]>

Use more samples for training of Profet model

Signed-off-by: Fabrice Normandin <[email protected]>

Warm-start example is a bit better

Signed-off-by: Fabrice Normandin <[email protected]>

Add more comments in the notebook

Signed-off-by: Fabrice Normandin <[email protected]>

Use Pytorch mnist/fashionmnist instead, bad

Signed-off-by: Fabrice Normandin <[email protected]>

[dirty] Trying to find good settings

Signed-off-by: Fabrice Normandin <[email protected]>

Update example a bit, seems to work much better!

Signed-off-by: Fabrice Normandin <[email protected]>

Try to make the plot appear

Signed-off-by: Fabrice Normandin <[email protected]>

Add the pickleddb files for the databases

Signed-off-by: Fabrice Normandin <[email protected]>

Rename things, image still does show :(

Signed-off-by: Fabrice Normandin <[email protected]>

Add a thumbnail (image shows up now!)

Signed-off-by: Fabrice Normandin <[email protected]>

Removed the torch multiprocessing context setting

Signed-off-by: Fabrice Normandin <[email protected]>

Make path relative so CI will work too

Signed-off-by: Fabrice Normandin <[email protected]>

Add tutorial to index, add text to describe plot

Signed-off-by: Fabrice Normandin <[email protected]>

Move databases to avoid any possible abs paths

Signed-off-by: Fabrice Normandin <[email protected]>

Attempt to fix KnowledgeBase config merge issue

Signed-off-by: Fabrice Normandin <[email protected]>

Update the pickle files for the example

Signed-off-by: Fabrice Normandin <[email protected]>

Seed all the randomness, update pickles

Signed-off-by: Fabrice Normandin <[email protected]>

Fix issue with portability of PickledDB objects

Signed-off-by: Fabrice Normandin <[email protected]>

Update the pickleddb files for the example

Signed-off-by: Fabrice Normandin <[email protected]>
@lebrice lebrice requested a review from bouthilx January 27, 2023 19:29
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