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

Do not round real-valued bounds #44

Closed
MLopez-Ibanez opened this issue Oct 24, 2022 · 0 comments
Closed

Do not round real-valued bounds #44

MLopez-Ibanez opened this issue Oct 24, 2022 · 0 comments

Comments

@MLopez-Ibanez
Copy link
Owner

MLopez-Ibanez commented Oct 24, 2022

The problem is apparently that this round() call sometimes results in irace seeing parameter bounds that are too wide; e.g. when the actual lower bound is 1e-5, then irace with the default readParameters digits value ends up calling the function with value 0. I am doing quite a lot of experiments currently and observed this often; maybe the mutator in irace tries the exact boundary value with nonzero probability? (Another example is lower bound log(2) (0.6931472) -> irace tries 0.6931, which is too low.)

We also accept some tolerance in our real-valued search space components, but that is just much tighter than four digits. I think we should call irace in a way that avoids rounding as much as possible.

Originally posted by @mb706 in mlr-org/bbotk#191 (comment)

MLopez-Ibanez added a commit that referenced this issue Oct 24, 2022
…(r) parameters would change depending on the value of `'digits'`.
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

1 participant