-
Notifications
You must be signed in to change notification settings - Fork 14
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
Suggestion fails because pareto groups is empty #6
Comments
Did you set |
Hmm, no, but I was editing the suggestions returned by CARBS. There's a round function for linear spaces but not for exponential ones, so I was postprocessing values to make minibatches powers of 2. Will see if I repro it in the cleaned up version |
@abefetterman Re-opening this because I found the cause of the issue, which appears to be a real bug When num_random_samples > 1, I was getting duplicate suggestions. This then causes group_observations to produce groups where the parameters are identical, but there are multiple different costs. But the quantile computation uses the min across all observations, not across groups. So there can be no groups for which the mean is less than the quantile. |
Hello, I got the same error, during both I did a quick fix and am testing it. If my runs work well, and it looks good to you, I'll make a PR. Thank you! |
I've got the same bug. Been running carbs for a few months with no issues, but getting this today. @kywch I will try your pr |
Hi @b-vm can you try https://github.com/kywch/carbs/tree/py310 this branch? Previously, I only changed BTW, |
Got CARBS working on a toy env. Went to throw it on something larger and it's failing after a few trials because the pareto front is empty. Any idea what would cause this?
The text was updated successfully, but these errors were encountered: