-
Notifications
You must be signed in to change notification settings - Fork 43
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
Tune schedule #974
base: main
Are you sure you want to change the base?
Tune schedule #974
Conversation
My high-level thoughts on
|
Note: let's take a look at how this structure can work with agua, which does the model fits en masse via h2o. |
Another note: We currently use the parameter set to determine which parameter, i.e., column in the grid, is associated with the preprocessing, model, or post-processing stage (via the |
For agua... the current routine for processing the grid normally is here and the agua equivalent is here. Basically, agua consolidates the work that For TL;DR agua will work similarly to tune, except instead of losing a loop, the loss of the model loop is offset by a post loop. It should still be very efficient. |
This draft PR is for testing a new data structure that governs how tune will efficiently process grids. Two new functions are currently exported for debugging but will not be exported in the final version:
get_tune_schedule()
produces a tibble structure with nested levels that contains information about what tuning parameters should be used at different stages of execution. This PR is focused on making sure that this structure accounts for all of the tuning scenarios.loopy()
is a temporary function that emulatestune_grid_loop_iter()
(without the bells and whistles) that is used to testget_tune_schedule()
and also to understand how we will incorporate postprocessors into the mix.