cmu-delphi/forecast-eval

Consider defining `USE_CURR_TRUTH` as a `reactive`

Ouverte

#273 ouverte le 25 avr. 2023

Voir sur GitHub
 (0 commentaire) (0 réaction) (0 personne assignée)R (3 forks)auto 404
good first issue

Métriques du dépôt

Stars
 (6 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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.

Guide contributeur

Direction de recherche
Identifiez où USE CURR TRUTH est actuellement défini (probablement comme reactiveVal ou reactiveValues). Remplacez ensuite l'observer qui le met à jour par une expression réactive qui se met automatiquement à jour lorsque ses dépendances changent. Assurez vous que tous les observers et sorties en aval lisent cette nouvelle valeur réactive au lieu d'être déclenchés par l'observer. Cela exploite l'évaluation paresseuse et peut améliorer les performances.
Stack technique
r
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
RShiny
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80