Skip to content

Commit

Permalink
remove dependency on dev yardstick (#694)
Browse files Browse the repository at this point in the history
Co-authored-by: Max Kuhn <mxkuhn@gmail.com>
  • Loading branch information
simonpcouch and topepo authored Jul 21, 2023
1 parent 2824085 commit c58bfb1
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
2 changes: 1 addition & 1 deletion DESCRIPTION
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ Imports:
vctrs (>= 0.6.1),
withr,
workflows (>= 1.0.0),
yardstick (>= 1.2.0.9001)
yardstick (>= 1.2.0)
Suggests:
C50,
censored,
Expand Down
5 changes: 5 additions & 0 deletions tests/testthat/test-grid.R
Original file line number Diff line number Diff line change
Expand Up @@ -132,6 +132,7 @@ test_that("tune model only (without recipe, multi-predict. #695)", {

test_that("tune model only (fairness - include `by` variable as predictor)", {
skip_on_cran()
skip_if_not_installed("yardstick", minimum_version = "1.2.0.9001")

knn <- parsnip::nearest_neighbor("classification", "kknn", neighbors = tune())
mtcars_fair <- mtcars
Expand Down Expand Up @@ -171,6 +172,7 @@ test_that("tune model only (fairness - include `by` variable as predictor)", {

test_that("tune model only (fairness - don't include `by` variable as predictor)", {
skip_on_cran()
skip_if_not_installed("yardstick", minimum_version = "1.2.0.9001")

knn <- parsnip::nearest_neighbor("classification", "kknn", neighbors = tune())
mtcars_fair <- mtcars
Expand Down Expand Up @@ -210,6 +212,7 @@ test_that("tune model only (fairness - don't include `by` variable as predictor)

test_that("tune model only (fairness metrics - evaluate across multiple `by`)", {
skip_on_cran()
skip_if_not_installed("yardstick", minimum_version = "1.2.0.9001")

knn <- parsnip::nearest_neighbor("classification", "kknn", neighbors = tune())
mtcars_fair <- mtcars
Expand Down Expand Up @@ -250,6 +253,7 @@ test_that("tune model only (fairness metrics - evaluate across multiple `by`)",

test_that("tune model only (fairness - evaluate across multiple `by`, same metric)", {
skip_on_cran()
skip_if_not_installed("yardstick", minimum_version = "1.2.0.9001")

knn <- parsnip::nearest_neighbor("classification", "kknn", neighbors = tune())
mtcars_fair <- mtcars
Expand Down Expand Up @@ -291,6 +295,7 @@ test_that("tune model only (fairness - evaluate across multiple `by`, same metri

test_that("tune model only (fairness - evaluate only fairness metrics)", {
skip_on_cran()
skip_if_not_installed("yardstick", minimum_version = "1.2.0.9001")

knn <- parsnip::nearest_neighbor("classification", "kknn", neighbors = tune())
mtcars_fair <- mtcars
Expand Down

0 comments on commit c58bfb1

Please sign in to comment.