biocore/empress

Save extra work of cos/sin calls

Ouverte

#225 ouverte le 29 juin 2020

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)JavaScript (32 forks)auto 404
good first issueperformancerefactoring

Métriques du dépôt

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

Description

@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.

Guide contributeur

Direction de recherche
Localisez les trois blocs de code mentionnés (empress.js lignes 404-420, 643-655 et tree.py lignes 437-449). Identifiez les appels répétés de cos/sin pour le même angle et remplacez les par une seule variable calculée, en réutilisant la valeur. Par exemple, calculez cos(angle) une fois et assignez le à une variable, puis utilisez cette variable plusieurs fois.
Stack technique
javascriptpython
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.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptPythonbasic trigonometry
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65