probcomp/curve-fitting

Turn opacity-scale into a multimethod that dispatches on mode

Aberta

#53 aberto em 27 de nov. de 2018

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

Métricas do repositório

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

Description

Currently we switch between opacity scales like so:

(let [make-opacity-scale (case mode
                           :resampling resampling/make-opacity-scale
                           :prior prior/make-opacity-scale)
      opacity-scale (make-opacity-scale (map :log-score curves))])

Instead we should define a multimethod curve-fitting.core/opacity-scale (or maybe curve-fitting.sketches/opacity-scale that takes two arguments: The mode and the log scores.

Guia do colaborador

Direção de pesquisa
Crie um multimétodo `curve fitting.core/opacity scale` que despache com base no argumento mode. Mova a lógica de seleção baseada em case para métodos separados para cada modo (por exemplo, :resampling e :prior). Atualize o chamador para usar o multimétodo em vez da expressão case.
Pilha de tecnologia
clojure
Domain
backenddata
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.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
ClojureGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65