-
-
Notifications
You must be signed in to change notification settings - Fork 985
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
It search only small parameter
area when using pruning.
#270
Comments
This code is patching to
It works as I intended but I don't know this is good way. |
parameter
area when started pruning.parameter
area when using pruning.
Thank you for reporting this issue. This is a known problem within our team. The combination of TPE and pruning sometimes poorly works, as distribution update is trapped by the pruning mechanism. This might be improved a bit at the today's release (if you are interested in the details, please see #268 and #261). Your patch would be a candidate of the solution, but it may cause some problems, e.g., the parameters suggested to the pruned trials are underestimated by TPE. What is the best solution is under consideration, and we will address it in the future. For a workaround, you can also try |
Thank you for response. I checked again with new version, It is improved this issue. Thank you. |
@higumachan FYI This problem was completely solved by #439 (the next release will include this patch). |
I encounted a problem of similar parameter on many trial when I'm using pruning.
I can not original code but I made similar case toy script bellow.
It running that output. (sorry)
I think that it seems that the last one repeats the similar parameter.
Does anyone have insight on this problem?
Or Is this the expected behavior?
The text was updated successfully, but these errors were encountered: