cmu-delphi/forecast-eval

Consider defining `USE_CURR_TRUTH` as a `reactive`

Aperta

#273 aperta il 25 apr 2023

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)R (3 fork)auto 404
good first issue

Metriche repository

Star
 (6 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

There are several observers 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 reactives are lazily evaluated, they are faster than observers.

Guida contributor

Direzione di ricerca
Identifica dove USE CURR TRUTH è attualmente definito (probabilmente come reactiveVal o reactiveValues). Quindi sostituisci l'observer che lo aggiorna con un'espressione reattiva che si aggiorna automaticamente quando le sue dipendenze cambiano. Assicurati che tutti gli observer e output downstream leggano da questa nuova espressione reattiva invece di essere attivati dall'observer. Questo sfrutta la valutazione pigra e può migliorare le prestazioni.
Tech stack
r
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
RShiny
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80