Skip to content

Commit

Permalink
remove model parameter from scoringutils call
Browse files Browse the repository at this point in the history
This change was introduced in
epiforecasts/scoringutils#915 and subsequently
broke our tests.
  • Loading branch information
zkamvar committed Oct 3, 2024
1 parent 372977e commit 0ab3702
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
3 changes: 2 additions & 1 deletion DESCRIPTION
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,8 @@ Authors@R: c(
role = c("aut")),
person(given = "Zhian",
family = "Kamvar",
email = "[email protected] ",
email = "[email protected]",
comment = c(ORCID = "0000-0003-1458-7108"),
role = c("ctb")),
person(given = "Becky",
family = "Sweger",
Expand Down
2 changes: 2 additions & 0 deletions tests/testthat/test-score_model_out.R
Original file line number Diff line number Diff line change
Expand Up @@ -393,6 +393,7 @@ test_that("score_model_out works with all kinds of interval levels are requested
regexp = "must be a number between 0 and 100"
)

suppressWarnings({
expect_warning(
score_model_out(
model_out_tbl = forecast_outputs |> dplyr::filter(.data[["output_type"]] == "quantile"),
Expand All @@ -419,6 +420,7 @@ test_that("score_model_out works with all kinds of interval levels are requested
),
"To compute the interval coverage for an interval range of" #scoringutils warning
)
})
})


Expand Down

0 comments on commit 0ab3702

Please sign in to comment.