angular-ui/ui-grid

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

Open

#6 960 ouverte le 19 avr. 2019

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)JavaScript (2 496 forks)batch import
good first issuegrid-coreseverity: performance

Métriques du dépôt

Stars
 (5 395 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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!

Guide contributeur

Direction de recherche
Identifiez toutes les utilisations de `forEach` et `map` sur les tableaux dans le code source de `ui grid`, effectuez des benchmarks de performance avec des outils comme jsben.ch et remplacez les par des alternatives plus rapides comme les boucles `for` ou des fonctions optimisées. Assurez vous de ne pas introduire de changements cassants.
Stack technique
javascriptangular
Domaine
frontend
Type d'issue
Performance
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
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
JavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65