tidymodels.org-legacy icon indicating copy to clipboard operation
tidymodels.org-legacy copied to clipboard

Implementing `update` method seems to be necessary for models to work with `tune`

Open dfalbel opened this issue 3 years ago • 0 comments

It's not easy to write a simple reprex for this, but you can comment out the update method of https://github.com/mlverse/tabnet create a model with at least one tune() and try to finalize it. I see an error with something like:

Error in update.default(object = list(args = list(lookback = ~100, horizon = ~4,  : 
  need an object with call component

Maybe this should be noted in: https://www.tidymodels.org/learn/develop/models/#add-parsnip-models-to-another-package

dfalbel avatar Apr 13 '22 17:04 dfalbel