cmu-delphi/forecast-eval

Consider defining `USE_CURR_TRUTH` as a `reactive`

Aberta

#273 aberto em 25 de abr. de 2023

Ver no GitHub
 (0 comentário) (0 reação) (0 responsável)R (3 forks)auto 404
good first issue

Métricas do repositório

Stars
 (6 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

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.

Guia do colaborador

Direção de pesquisa
Identifique onde USE CURR TRUTH está atualmente definido (provavelmente como reactiveVal ou reactiveValues). Em seguida, substitua o observer que o atualiza por uma expressão reativa que atualiza automaticamente quando suas dependências mudam. Certifique se de que todos os observers e outputs downstream leiam desta nova expressão reativa em vez de serem acionados pelo observer. Isso aproveita a avaliação preguiçosa e pode melhorar o desempenho.
Pilha de tecnologia
r
Domain
backend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
RShiny
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80