-
Notifications
You must be signed in to change notification settings - Fork 25
Rework score() #344
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Rework score() #344
Changes from all commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
a3b0753
Update input check functions to adhere to checkmate structure with ch…
nikosbosse aeb725c
Add functions to test the forecast type of the data used as input for…
nikosbosse b299331
move global variables to its own file
nikosbosse e3f6b54
Create a function, `validate()`, that replaces `check_forecasts()` an…
nikosbosse e30da91
Remove excessive checks from get_forecast_type, create new function c…
nikosbosse d847ae8
fix input checks for scoring functions
nikosbosse 0a96dd1
remove check_forecasts.R, move input check helpers to new file, creat…
nikosbosse 346eb54
Use validate() everywhere instead of check_forecasts
nikosbosse d67446a
update utils file with get_prediction_type and get_forecast_unit
nikosbosse 8d0bf69
update documentation
nikosbosse 2812b21
Update tests
nikosbosse 603d042
update snapshots - apparently the order of models has changed for som…
nikosbosse 353f07f
Automatic readme update
actions-user 2f0990b
Reorganise files such that scores can be found in files called "metri…
nikosbosse 8510598
update documentation for moved metrics
nikosbosse ee2a94c
Reorganise files for input check functions and helpers
nikosbosse b573e82
remove unneeded check functions
nikosbosse aa112aa
Renamed `find_duplicates()` to `get_duplicate_forecasts()`, reorganis…
nikosbosse e0648a1
Update news.md
nikosbosse 1207d2a
Add tests for get_duplicate_forecasts, update documentation
nikosbosse 9c8447c
Create / update example data for point forecasts
nikosbosse 7b19334
fix point forecast example data
nikosbosse 8ffed2e
change error to a message in `mad_sample()` if `observed` is provided
nikosbosse 679bed9
Rewrite `score()` as S3 method. score.scoringutils_quantile() still m…
nikosbosse 0a93c6e
update tests for score(), delete old files
nikosbosse 3ad29c1
Remove message in mad_sample()
nikosbosse 83b7589
Replace metrics argument in score by a list of functions
nikosbosse 3b4b742
Create documentation for metrics_ data
nikosbosse a202e0c
Handle `...` function arguments that get passed through `score()` mor…
nikosbosse c005b14
Create first version of a function to validate metric inputs
nikosbosse e31d100
Add tests for the validate_metrics function
nikosbosse caf3ecc
Address issues raised by Sam: rename geom_mean() --> geometric_mean()…
nikosbosse 36025b3
Merge branch 'rework-lower-level-fcts' into rework-score()
nikosbosse 52640c9
Remove "clever" code to fix unnamed metrics argument and restrict it …
nikosbosse 42d55fe
update tests to reflect change that metrics need to be a named list now
nikosbosse 4689e68
Update some more documentation
nikosbosse 09b941e
Merge branch 'scoringutils-review' into rework-score()
nikosbosse File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -53,6 +53,7 @@ Imports: | |
| ggplot2 (>= 3.4.0), | ||
| lifecycle, | ||
| methods, | ||
| Metrics, | ||
| rlang, | ||
| scoringRules, | ||
| stats | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.