Simon P. Couch

Results 151 issues of Simon P. Couch

In: https://github.com/tidymodels/hardhat/blob/ed8b0328b59d4e6a90d79605cc760978799e39df/R/mold.R#L32-L33 The slot should be `outcomes` rather than `outcome`.

documentation
tidy-dev-day :nerd_face:

Related to [tidymodels/tune#660](https://github.com/tidymodels/tune/issues/660#issuecomment-2035486249). It looks like, currently, `tune_args()` doesn't include arguments marked for tuning if they're not tunable. ``` r library(tidymodels) rec % step_impute_bag( Status, Home, Marital, Job, Income, Assets,...

bug

Because of a partial revert in #135 there's now an unused helper in the source: https://github.com/tidymodels/probably/blob/f36f4b20f2812551dbca87da9f4d92f98e262b6d/R/cal-estimate-utils.R#L214-L217

tidy-dev-day :nerd_face:
upkeep

When making patches to fix runs on `general-meta`, I noticed that multisession workers required that needed parsnip extension packages be loaded via `options.future` inside of `fit_members()`: https://github.com/tidymodels/stacks/blob/f2e33419b5712ae7df3ca4d689c652188dce4b59/R/fit_members.R#L157 I wonder if...

`use_standalone("r-lib/rlang", "types-check")` preferred over home-grown argument checkers.

upkeep

Some explanation of the internal data structures and how different senses of train/test splits are utilized in the package—finding that it would be nice to point to something like this....

documentation 📜

tips/tricks to stack models quickly for preliminary results—racing / choosing grids, choosing model specs, possible results of #119

documentation 📜

Related to https://github.com/tidymodels/workflows/pull/225, https://github.com/tidymodels/container/pull/12. This PR proposes resampling calibrators using an "internal split"—it's _very_ scrappy at the moment and intended only for internal testing. ``` r library(tidymodels) library(container) library(probably) #>...

In `tune_grid_loop_iter()`, we pass `split` to several error handling functions that only use the split to generate informative error messages, i.e. with `labels(split)`. We should instead just write: ```r split_label

upkeep

We recently discussed possible alternative interfaces for defining information on tuning parameters. Changes to make this happen would happen in many packages, though the main context where folks would encounter...

feature