Closed nikosbosse closed 5 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 95.29%. Comparing base (
8534795
) to head (f70bf30
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description
As discussed in #447, some functions lack input checks. This is the third PR that adds more checks.
This PR
summarise_scores()
get_metrics()
+ associated testsmerge_pred_and_obs()
sample_to_quantile()
run_safely()
data.table::
calls withensure_data.table()
Further thoughts:
sample_to_quantile()
should require a validatedforecast_sample
. I opened a separate issue for this: https://github.com/epiforecasts/scoringutils/issues/755Checklist
lintr::lint_package()
to check for style issues introduced by my changes.