-
-
Notifications
You must be signed in to change notification settings - Fork 5
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
hyperband and CV. is this combinable? #5
Comments
It's already working as it is by replacing holdout with cv.
I think it makes great sense as we are less likely to waste computing power/time on "lucky" configs in advanced bracket stages, plus the usual CV benefits. An example with 3-fold CV:
|
@SebGruber1996 simply make sure that this work also in a unit test (with a small CV) then we are done here |
added 2 fold CV to the unit tests |
One other question would be if we can for example only do |
I opened a new issue for that #56. |
There was a bug regarding the evaluation of configs with cross-validation #50. Tests did not cover this case, so maybe tests should be improved. |
No description provided.
The text was updated successfully, but these errors were encountered: