Simon P. Couch

Results 151 issues of Simon P. Couch

Currently, tune uses [foreach](https://cran.r-project.org/web/packages/foreach/vignettes/foreach.html) to facilitate parallel computing. As a group, we've discussed gradually deprecating our support for foreach in favor of the [futureverse](https://www.futureverse.org/packages-overview.html). The big ideas: * future does...

breaking change :skull_and_crossbones:
feature

Much of tune's core internals were written >4yrs ago, before the introduction/maturation of more principled tooling for checking, logging, and erroring from cli and rlang, and before the tidymodels packages...

upkeep

The trace back for dropping into `tune_grid_loop_iter()`, the layer of the grid code path that, for a given resample, carries out the preprocessor and model fitting, looks something like this:...

upkeep

When unnamed arguments are passed to `...`, the warning message is a bit 🤨 ``` r library(tidymodels) # mistakenly forget to pass metrics argument name fit_resamples( linear_reg(), mpg ~ .,...

bug

Opening as a new issue so that this doesn't get lost when closing #409. We may be able to use the machinery in `stacks:::.get_glmn_coefs()` here. :) _Originally posted by @hfrick...

upkeep

`fit()`ting a single model vs resampling a model fit via `fit_resamples()` has a nice parallelism to it with the usual single-model approach: ``` r # single model: fit(workflow, data) #...

feature

The `initial` argument to `tune_bayes()`, `tune_sim_anneal()`, etc(?) can be expressed as either 1) a previous tuning result or 2) a number indicating the number of initial grid points to evaluate....

feature

Before https://github.com/tidymodels/tune/pull/575, failing `extract` code used to raise no prompts to the user. The test objects try to tidy kknn objects in `extract` code and result in errors. They're now...

upkeep

`tune_bayes()`' functionality for processing a non-`tune_result` `initial` input: https://github.com/tidymodels/tune/blob/f6300b1e2f3c97ef2a4f48e2aa62b8c64cb238ac/R/tune_bayes.R#L490-L499 Duplicates checks and grid generation from `tune_grid()`'s `check_grid`, which it drops into anyway. https://github.com/tidymodels/tune/blob/f6300b1e2f3c97ef2a4f48e2aa62b8c64cb238ac/R/checks.R#L116

upkeep

We ought to write out the`val_class_and_single()` type checkers in favor of rlang's standalone type checkers. Their checkers give us finer-grained support for testing classes, are more resilient to edge cases,...

upkeep