DEHB
DEHB copied to clipboard
automl
Continue an optimization run.
This PR adds the functionality to continue an optimization run.
Also, it adds the possibility to restrict the worker to only run a single task concurrently. It happened that a worker was overloaded with tasks, however, due to mem limitations it shouldn't perform multiple tasks in parallel. This solution is similar to this StackOverflow article
I will add a example on how to use restarting and how to limit the resources per worker.
@Neeratyoy: Could you please create a dev branch, so I don't have to push to your main branch. This would make me sleep better :-)
Could you please create a dev branch, so I don't have to push to your main branch
done :)
Could you please create a dev branch, so I don't have to push to your main branch
done :)
Perfect. Is now pointing to dev.
This is actually a great contribution so thanks @PhMueller
However, if I may ask you to do some extra work or to have your permission if I do so
I was thinking of having one PR with the __version__ and setup to allow pip install for DEHB and that can be merged with master asap
as for the other main change in this PR, I'd like to understand the change a bit better and standardize a few things
for instance, to understand client_resources better we should include a docstring I think, which would necessitate docstrings for the other arguments too, which I would like to add anyways
so please let me know if your work can proceed with the forked version, and if you can split this PR, or if you give me a green flag I can do that by copying your files, but it's only fair that you're the author for that PR ;)
Hi, @PhMueller would it be possible to include the commit https://github.com/automl/DEHB/pull/6/commits/fc5fd98509b872358c752095341fed55671d7f6e for pip installability in a separate PR? Because, currently @Neeratyoy does not have time to look at all the issues in this PR. And being installable by pip is useful anyway, e.g., for me the example here: https://github.com/automl/DEHB/tree/master/utils seems to require dehb to be installed.
Hey,
Sorry for neglecting this pr. I will hopefully find some time to finish this one end of this year.
In the meanwhile: @RaghuSpaceRajan, you can take a look at #11.