biocore/empress

Save extra work of cos/sin calls

Offen

#225 geöffnet am 29.06.2020

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (32 Forks)auto 404
good first issueperformancerefactoring

Repository-Metriken

Stars
 (56 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Contributor Guide

Research-Richtung
Finden Sie die drei genannten Codeblöcke (empress.js Zeilen 404-420, 643-655 und tree.py Zeilen 437-449). Identifizieren Sie wiederholte cos/sin Aufrufe für denselben Winkel und ersetzen Sie sie durch eine einzelne berechnete Variable, wobei Sie den Wert wiederverwenden. Berechnen Sie z.B. cos(angle) einmal und weisen Sie es einer Variablen zu, verwenden Sie diese Variable dann mehrfach.
Tech Stack
javascriptpython
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScriptPythonbasic trigonometry
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65