Michael Mahoney

I think spatialsample is ready for version 1.0.0. We're building on top of rsample >= 1.0.0, which has had a pretty stable interface for multiple years, and I'd be surprised...

GIFs in the Buffering article have not updated to the new autoplot

4

Both the main and dev site still use the old (fully opaque) autoplot method for buffering GIFs: https://spatialsample.tidymodels.org/dev/articles/buffering.html I'm not entirely sure why that's happening; I guess we need to...

Function for returning the original data frame with fold assignments appended?

1

## Feature Over on spatialsample, there have been a few requests (https://github.com/tidymodels/spatialsample/issues/158, https://github.com/tidymodels/spatialsample/issues/157) for a function that basically works like this: ``` r library(rsample) library(magrittr) library(generics) augment.rset Tree age circumference...

Could `make_strata()` warn (or remove the `strata` attribute) when only returning a single strata? Or message when pooling at all?

1

## Feature I was reminded about #438 by the GitHub lock bot, an issue where a user was surprised that `vfold_cv()` (and eventually `make_strata()`) "didn't stratify" (or rather, treated the...

Is there a way to get (specific columns of the) modeled data for metric calculations?

1

## Feature For loss functions which require additional data sources, is there a way to access the input data (or perhaps just specific columns) while tuning? For example, the experimental...

Could `get_rsplit()` add a method for `resample_results` from `tune::fit_resamples()`?

1

## The problem In answering https://github.com/tidymodels/hardhat/issues/240 , I was tempted to use the `rsample::get_rsplit()` helper function on the output from `tune::fit_resamples()` to avoid needing to use the `$splits[[1]]` syntax, which...

`tune::collect_metrics()` sorts metrics in a different order than `yardstick::metrics_set()`

1

## The problem With current HEAD, `tune::collect_metrics()` appears to return metrics in alphabetical order, while `yardstick::metrics_set()` returns metrics in the order they were initially provided. This isn't a big deal,...

Workflows which use case weights calculated in a recipe from non-predictor columns error when attempting to `predict()`.

1

## The problem Workflows which use dynamic case weights (calculated from non-predictor columns) error when attempting to `predict()`. This is following a string of issues, most recently https://github.com/tidymodels/hardhat/issues/240 , and...

`metric_set()` has a less useful error message when no `estimate` is provided for classification metric sets

1

## The problem For classification metrics, when `truth` or `estimate` is not a factor, `metric_set()` gives a much less informative error than the classification metric itself. ## Reproducible example ```...

Check type of `na_rm`?

2

## Feature Right now, passing anything that's truthy to `na_rm` will remove NA values, while values that can't be casted to logical crash in an `if` statement: ``` r yardstick::rmse_vec(c(1,...