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

Suggestion fails because pareto groups is empty #6

Open
jsuarez5341 opened this issue Jun 26, 2024 · 6 comments
Open

Suggestion fails because pareto groups is empty #6

jsuarez5341 opened this issue Jun 26, 2024 · 6 comments

Comments

@jsuarez5341
Copy link

image

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?

@abefetterman
Copy link
Contributor

Did you set num_random_samples in your config to 0? That's the only thing that I imagine could cause it -- we recommend setting it to at least 3 (if you are running without any parallel suggestions, set it to at least the number of dimensions you have)

@jsuarez5341
Copy link
Author

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

@jsuarez5341
Copy link
Author

@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.

@kywch
Copy link

kywch commented Aug 20, 2024

Hello, I got the same error, during both suggest() and observe().

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!

main...kywch:carbs:main

@b-vm
Copy link

b-vm commented Nov 9, 2024

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

@kywch
Copy link

kywch commented Nov 14, 2024

Hi @b-vm can you try https://github.com/kywch/carbs/tree/py310 this branch?

Previously, I only changed get_pareto_groups_conservative but your error comes from get_pareto_groups, due to a similar cause.

BTW, resample_frequency=2 gives you the previous suggestion once every two suggestions. Is that what you intended?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants