Skip to content

Conversation

JedGrabman
Copy link
Contributor

Uses the new version of get_covidhub_predictions by querying multiple forecasters at once.
Also makes use of evaluate_covid_predictions, by evaluating multiple signals at once.

covidcast also now returns actual data for territories, so CovidHubUtils only needs to be used for national level data.

Deletes predictions.R
Refactor score.R

@kateharwood
Copy link
Contributor

This does not adjust the memory usage as far as you know, right? Maybe we should test this with the deploy workflow to make sure everything still runs through?

@JedGrabman
Copy link
Contributor Author

As far as I know, there shouldn't be a memory impact, but I kicked off the deploy workflow to test it: https://github.com/cmu-delphi/forecast-eval/runs/2222184360?check_suite_focus=true

@kateharwood kateharwood merged commit 66b72b1 into dev Mar 30, 2021
@kateharwood kateharwood deleted the dev-pred-simplify branch March 30, 2021 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants