Use brulee::brulee_activations() to check for available activation functions in parsnip::check_args.mlp()
#1019
Labels
feature
a feature request or enhancement
Comments
|
Thanks for flagging this @christophscheuch ! We'll get this opened up once the dust of the current work on {brulee} by @topepo has settled a bit. |
|
I am working on this now that I just sent brulee to CRAN (with those features). |
topepo
pushed a commit
that referenced
this issue
Feb 14, 2024
Merged
topepo
added a commit
that referenced
this issue
Feb 14, 2024
* remove mlp activation check for #1019 * unit test for #1019 * update news * version requirement in skip --------- Co-authored-by: ‘topepo’ <‘[email protected]’>
|
This issue has been automatically locked. If you believe you have found a related problem, please file a new issue (with a reprex: https://reprex.tidyverse.org) and link to this issue. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
A recent change in
bruleeintroduced new activation functions andbrulee_activations(), which provides available functions (see PR #74). However,parsnipdoes not support these new activation functions yet, becauseparsnip::check_args.mlp()has hard coded values (see function definition here).Would be great to have support for the new activation functions in parsnip.
The text was updated successfully, but these errors were encountered: