cmu-delphi / cmu-delphi/forecast-eval

Consider defining `USE_CURR_TRUTH` as a `reactive`

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

Description

There are [several `observer`s](https://github.com/cmu-delphi/forecast-eval/blob/49896afaece66985f0b0d59c3eab1dee20bfe32a/app/server.R#L814-L820) that exist only to change the value of `USE_CURR_TRUTH`. The logic could be simpler and dashboard faster if `USE_CURR_TRUTH` were changed into a `reactive`. Since `reactive`s are lazily evaluated, they are faster than `observer`s.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start in app/server.R around lines 814-820 and inspect the observers that update USE_CURR_TRUTH. Confirm how the dashboard currently responds to those changes, then verify that the reactive-based behavior preserves the existing dashboard results and improves the relevant update flow.

Written by the indexing model from the issue text.

Assessment

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.