Simon P. Couch

Results 165 comments of Simon P. Couch

With an eye for reducing `Remotes` hoopla, I'm going to go ahead and merge and open issues for smaller todos.

Long time no see😝 We've got some good news here, though—custom probability thresholds and other postprocessing functionality is now available via [tailor](https://github.com/tidymodels/tailor)s, which can be added to workflows in the...

You may interested in [`new_groupwise_metric()`](https://yardstick.tidymodels.org/reference/new_groupwise_metric.html) from yardstick 1.3.0. :) The new ["grouping behavior in yardstick"](https://yardstick.tidymodels.org/articles/grouping.html#groupwise-metrics) vignette describes what we mean by "groupwise." I believe we may be able to close...

I don't see a trivial way of incorporating this. It seems like [`advanced-01`](https://github.com/tidymodels/workshops/blob/main/slides/advanced-01-feature-engineering.qmd#L613) acknowledges that this is an issue for both variables and chooses to use `agent` as the variable...

I would say that change ought to be accompanied by adjustments to the visualizations in "Per-agent statistics" that provide context / intuition in the preceding slides, though, and that seems...

Similar request on [SO](https://stackoverflow.com/questions/75817057/how-to-parallelize-workflowsetsworkflow-map).

If this ever comes to the top of our to-do list, worth reading "Nested parallelism and protection against it' in [Bengtsson (2021)](https://journal.r-project.org/archive/2021/RJ-2021-048/RJ-2021-048.pdf).

Ah, I just came across `update_workflow_model()` while making some other changes, which does provide an API to pass a model formula to workflow sets! Maybe this could be better documented?

Do you anticipate any downsides to, instead, just fully switching to registering `kernlab::ksvm()` via its XY interface? Same question would go for `coxnet_train()` as well, I guess.