angular-ui/ui-grid

[Performance Increase] Replace prototype forEach/map within ui-grid for better performance

Geschlossen

#6.960 geöffnet am 19.04.2019

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (2.496 Forks)batch import
good first issuegrid-coreseverity: performance

Repository-Metriken

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

Beschreibung

http://jsben.ch/N3AVA

Looking at the performance of the faster ForEach/map functions written you can see that significant performance increase happens over the built in forEach/map prototype on a javascript array.

I saw that ui-grid uses the built in prototypes and figured they could use a performance increase so that ui-grid becomes more responsive with larger grid sizes. The larger the array, the more significant of a performance increase it has!

Contributor Guide

Research-Richtung
Identifiziere alle Verwendungen von `forEach` und `map` auf Arrays im Quellcode von `ui grid`, führe Performance Benchmarks mit Tools wie jsben.ch durch und ersetze sie durch schnellere Alternativen wie `for` Schleifen oder optimierte Funktionen. Stelle sicher, dass keine brechenden Änderungen vorgenommen werden.
Tech Stack
javascriptangular
Domain
frontend
Issue Type
Performance
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
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
JavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65