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

Restart Hyperopt from pickle tries.pkl file #1815

Closed
Cmurilochem opened this issue Oct 12, 2023 · 1 comment · Fixed by #1824
Closed

Restart Hyperopt from pickle tries.pkl file #1815

Cmurilochem opened this issue Oct 12, 2023 · 1 comment · Fixed by #1824
Assignees
Labels
question Further information is requested

Comments

@Cmurilochem
Copy link
Collaborator

In restart_hyperopt and PR #1812, I've made some changes to allow us conduct hyperoptimization restarts. By adding some extra methods to the FileTrials class, we are for the moment able to restart jobs using a pickle tries.pkl file that is currently used to:

  • save final states of FileTrials after all trials in case of a normal job.
  • initiate the state of FileTrials to that of a previous run in case of restarts, e.g, using initial_filetrials = FileTrials.from_pkl(tries.pkl).

Would this fulfil our requirements for the moment ? I am also currently trying to add an extra @classmethod to the FileTrials class, e.g., filetrials = FileTrials.from_json(tries.json, *args, **kwargs) that could also be used for restarting in addition to tries.pkl. But I am facing some initial difficulties due to the special formatting and encodings present in tries.json. Not sure if the restart from json would possible or even sustainable as this could no longer work in case of any future changes in our current json structure. Please, I would appreciate any suggestions or comments.

@Cmurilochem Cmurilochem added the question Further information is requested label Oct 12, 2023
@Radonirinaunimi
Copy link
Member

Hi @Cmurilochem, will you be in Science Park tomorrow (or early next week) such that we can discuss how this should be exactly done? Otherwise, zoom will also be fine.

@Cmurilochem Cmurilochem linked a pull request Oct 24, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants