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

use an initial random design with grid size is 1 #962

Open
topepo opened this issue Nov 13, 2024 · 0 comments · May be fixed by #965
Open

use an initial random design with grid size is 1 #962

topepo opened this issue Nov 13, 2024 · 0 comments · May be fixed by #965
Labels
feature a feature request or enhancement

Comments

@topepo
Copy link
Member

topepo commented Nov 13, 2024

tune:::create_initial_set() will use dials::grid_space_filling() when an integer is passed.

We have space-filling designs for 2+ points and when grid = 1, it creates a random grid but uses a LHD and issues a warning:

Due to the small size of the grid, a Latin hypercube design will be used.

We can get a random point via grid_random() without the warning.

finetune::tune_sim_anneal() defaults to a 1-point initialization so no warning would be good.

@topepo topepo added the feature a feature request or enhancement label Nov 13, 2024
topepo added a commit that referenced this issue Nov 13, 2024
@topepo topepo linked a pull request Nov 14, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature a feature request or enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant