Skip to content

Commit

Permalink
Updates for brulee_mlp_two_layer (#84)
Browse files Browse the repository at this point in the history
* new function for two layer models

* two layer unit tests

* avoid overflow issues

* fix doc error

* test when loss cannot be computed

* tests for tunable values

* update GHA

* update spelling

* use current CRAN torch

* add missing snapshot and remove print snapshots (due to OS differences)

* don't test on R 4.1

* test overflow on M1 mac

* small update

* finalize tunable for 2layer networks

* finalize tunable for 2layer networks

* remove tunable info (moved to parsnip)

* remove test file

---------

Co-authored-by: ‘topepo’ <‘[email protected]’>
  • Loading branch information
topepo and ‘topepo’ authored Sep 9, 2024
1 parent 0f90be5 commit 478a52c
Show file tree
Hide file tree
Showing 7 changed files with 5 additions and 162 deletions.
2 changes: 2 additions & 0 deletions .github/workflows/test-coverage.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,8 @@ name: test-coverage.yaml

permissions: read-all

permissions: read-all

jobs:
test-coverage:
runs-on: ubuntu-latest
Expand Down
2 changes: 1 addition & 1 deletion DESCRIPTION
Original file line number Diff line number Diff line change
Expand Up @@ -40,4 +40,4 @@ Config/testthat/edition: 3
Encoding: UTF-8
Language: en-US
Roxygen: list(markdown = TRUE)
RoxygenNote: 7.3.1
RoxygenNote: 7.3.2
5 changes: 0 additions & 5 deletions NAMESPACE
Original file line number Diff line number Diff line change
Expand Up @@ -41,11 +41,6 @@ S3method(print,brulee_linear_reg)
S3method(print,brulee_logistic_reg)
S3method(print,brulee_mlp)
S3method(print,brulee_multinomial_reg)
S3method(tunable,brulee_linear_reg)
S3method(tunable,brulee_logistic_reg)
S3method(tunable,brulee_mlp)
S3method(tunable,brulee_mlp_two_layer)
S3method(tunable,brulee_multinomial_reg)
export("%>%")
export(autoplot)
export(brulee_activations)
Expand Down
121 changes: 0 additions & 121 deletions R/tunable.R

This file was deleted.

25 changes: 0 additions & 25 deletions man/brulee-internal.Rd

This file was deleted.

4 changes: 2 additions & 2 deletions tests/testthat/_snaps/tunable.md
Original file line number Diff line number Diff line change
Expand Up @@ -336,7 +336,7 @@
[1] "dials"
[[3]]$fun
[1] "hidden_units"
[1] "hidden_units_2"
[[3]]$range
[1] 2 50
Expand All @@ -358,7 +358,7 @@
[1] "dials"
[[5]]$fun
[1] "activation"
[1] "activation_2"
[[5]]$values
[1] "relu" "tanh" "elu" "log_sigmoid" "tanhshrink"
Expand Down
8 changes: 0 additions & 8 deletions tests/testthat/test-tunable.R

This file was deleted.

0 comments on commit 478a52c

Please sign in to comment.