recipes

recipes copied to clipboard

rewrite fun_calls() to not use recursion

1

I learned about `all.names()` a couple of days ago. And it works wonders in recipes! The previous issues we had with long formulas was that `fun_calls()` was recursive, and it...

EmilHvitfeldt

``` r library(tidymodels) set.seed(12345) dat % step_interact(~ starts_with("race_"):starts_with("hisp_") + starts_with("race_"):starts_with("cat")) %>% prep() colnames(bake(rec, new_data = dat)) #> [1] "outcome" "race_o" "race_w" #> [4] "hisp_nhisp" "cat_b" "cat_c" #> [7] "cat_d" "cat_e"...

EmilHvitfeldt

Add an option, perhaps called `.pkgs`, and update the `required_pkgs()` method for this step. See [this post](https://community.rstudio.com/t/adding-packages-to-the-recipes-required-pkgs/181216)/

topepo

offset role

1

There's a new package on CRAN called `offsetreg`, which extends `tidymodels` with some additional model types and engines that use an offset, i.e., a variable which should always have a...

jdonland

Long formulas cause `recipe()` to error due to C stack usage

2

## The problem Long formulas (with many terms; I don't _believe_ the length of column names matters) cause `recipe()` to error due to C stack usage. Apologies if this is...

mikemahoney218

Document how to add arguments to steps without backwards breakage

1

what the title says. I forget too often that I should do this

EmilHvitfeldt

step_interact documentation is not correct

1

For documentation (see page below), the argument `keep_original_cols` defaults to `TRUE` not `FALSE`. Of course, I personally prefer `TRUE` as the default behavior. https://recipes.tidymodels.org/reference/step_interact.html

millermc38

audit use of ... in package

1

The `...` are used throughout the package for different things. Many of the steps define them themselves where it would probably be better if they were to be inherited. Other...

EmilHvitfeldt

right now deprecated functions live alongside the other files. we should follow the style guide https://style.tidyverse.org/package-files.html and move those into their own files to avoid developer confusion

EmilHvitfeldt

A number of steps use a `options` argument, which is supposed to be a named list of argument, that we pass to an underlying function. This should be refactored for...

EmilHvitfeldt