probcomp/curve-fitting

Turn opacity-scale into a multimethod that dispatches on mode

Aperta

#53 aperta il 27 nov 2018

Vedi su GitHub
 (0 commenti) (0 reazioni) (1 assegnatario)Clojure (3 fork)auto 404
good first issuerefactor

Metriche repository

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

Descrizione

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.

Guida contributor

Direzione di ricerca
Crea un multimethod `curve fitting.core/opacity scale` che dispacci sul parametro mode. Sposta la logica di selezione basata su case in metodi separati per ogni modalità (ad esempio :resampling e :prior). Aggiorna il chiamante per utilizzare il multimethod invece dell'espressione case.
Tech stack
clojure
Dominio
backenddata
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
ClojureGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65