Simon P. Couch

Results 156 issues of Simon P. Couch
trafficstars

Seeing the following deprecation condition while testing tailor: ``` transpose() is superseded. Please use `list_transpose()` instead. ``` Traceback: ``` 8. ├─generics::fit(tlr, d_calibration, outcome = y, estimate = y_pred) 9. ├─tailor:::fit.tailor(tlr,...

A little bit of developer complexity for the sake of participant simplicity. Given that we're already teaching a binary classification problem, I thought it'd be nice if the colors matched...

A revival of #479, reprex whittled down from tidymodels/finetune#116, will also fix tidymodels/yardstick#514 when addressed. When using socket cluster parallelism (notably, this not an issue with forking), workers can't find...

bug

Closes #255. Opted to just check for non-`NULL` arguments rather than missingness to allow for passing arguments programmatically more easily. "Does not require training" also doesn't feel quite informative enough,...

Same goes for `add_tailor(method)`: if the workflow doesn't require an inner split but arguments are passed related to it, we ought to error in `add_tailor()`.

When the rsample function called internally by `inner_split()` methods doesn't have a `prop` argument, that function errors. e.g. with `method = "boot_split"`: ``` r library(tidymodels) library(tailor) tlr % adjust_numeric_calibration() %>%...

Currently just reprints the tailor, but may be worth displaying the proportion and method when printing the workflow.

Related to #239—just a place to keep notes on the thought process for supporting sparse tibbles with formula preprocessors. In #245, we see: ``` r library(tidymodels) sparse_hotel_rates

Related to #1200--the package is now back on CRAN.