cmu-delphi / cmu-delphi/forecast-eval

Drop unused county hospitalization aheads early in pipeline instead of in dashboard

Open
#264 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
R
Stars
6
Forks
3
PR merge metrics
No merged PRs in 30d

Description

Hospitalizations forecasts include predictions for 1-28 days ahead, versus the 1-4 weeks ahead used for cases and deaths. However, in the dashboard we actually display only 4 of the 28 aheads for hospitalizations, so we waste a lot of time in the pipeline loading and scoring, and in the [dashboard filtering](https://github.com/cmu-delphi/forecast-eval/blob/b81379936d2e88bf37f90a9fd3100d4dbe3888ad/app/R/data_manipulation.R#L58) those extra aheads.

The extra aheads were originally included in the pipeline as a "just in case" but haven't been useful yet.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start in app/R/data_manipulation.R at the linked dashboard filtering code, then trace where hospitalization aheads are loaded and scored earlier in the pipeline. Done means the unused hospitalization aheads are removed before those stages while the dashboard still displays the intended four aheads.

Written by the indexing model from the issue text.

Assessment

Tech stack
r
Domain
analytics, data
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.