biocore/empress

Save extra work of cos/sin calls

Aperta

#225 aperta il 29 giu 2020

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (32 fork)auto 404
good first issueperformancerefactoring

Metriche repository

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

Descrizione

@wasade pointed this out about this JS code -- there are 8 calls to cos or sin within this block, but we could cut this down to just 4 calls by saving the value of each call for later in the block.

From some more digging, this also impacts this JS code for drawing thick lines, as well as this python code.

Guida contributor

Direzione di ricerca
Individua i tre blocchi di codice menzionati (empress.js righe 404-420, 643-655 e tree.py righe 437-449). Identifica le chiamate ripetute a cos/sin per lo stesso angolo e sostituiscile con una singola variabile calcolata, riutilizzando il valore. Ad esempio, calcola cos(angle) una volta e assegnalo a una variabile, quindi usa quella variabile più volte.
Tech stack
javascriptpython
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.
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
JavaScriptPythonbasic trigonometry
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65