Simon P. Couch
Simon P. Couch
Ah, looks like we'd expect to see a few breakages with the new xgboost: https://github.com/dmlc/xgboost/issues/9810. `watchlist` deprecated as an `xgb.train()` argument and `info` as a `xgboost::xgb.DMatrix()` argument. We will address...
Noting that this is likely related to / duplicate of https://github.com/tidymodels/parsnip/issues/1087. :)
Noting a tag in https://github.com/dmlc/xgboost/issues/9810#issuecomment-2599763233 that has some higher-level information.
Related/duplicate of https://github.com/tidymodels/tune/issues/829!
I appreciate you bringing this up, @AmeliaMN! For now, `fit.infer` just applies `tidy()` to the output of each `glm()` fit. We could maybe open up an interface here for passing...
Spent a bit with 4c763ed in an attempt to enable the submodel trick but still not quite there. None of the tuning code now works from this PR as I...
By "argument" do you mean we should add an argument to `axe_*.workflow` methods to toggle whether to additionally butcher the calibration objects or just that butchering a workflow ought to...
I would make the argument here that, since the user has explicitly chosen with a non-default argument setting `save_workflow = TRUE` to store that object, we ought not to warn...
Test ref: https://github.com/tidymodels/workflows/blob/1a3d7eb001ba424c6ed81c91606054448e82f28f/tests/testthat/test-fit.R#L246-L247
More thoughts from Hannah in a follow-up comment on the same PR: > After chatting with Max: > > * Max agrees with the basic assumptions except for the single...