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.

Sign up for GitHub

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

Jump to bottom

`regularization_factor()` should be exclusive of 0 #381

Closed

hfrick opened this issue Feb 14, 2025 · 0 comments · Fixed by #382

Member

hfrick commented Feb 14, 2025

# dials::regularization_factor() should be exclusive of 0
fails <- ranger::ranger(Species ~ ., data = iris, regularization.factor = 0)
#> [...]
#> Error in ranger::ranger(Species ~ ., data = iris, regularization.factor = 0): 
#> The regularization coefficients cannot be smaller than 0.

# keeping 1 as inclusive is okay
runs <- ranger::ranger(Species ~ ., data = iris, regularization.factor = 1)

^{Created on 2025-02-14 with reprex v2.1.1}

hfrick added a commit that referenced this issue


          fix for #381

44802c5

hfrick mentioned this issue

regularization_factor() exclusive of lower border #382

Merged

hfrick closed this as completed in #382

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment