Résolution du bug des fichiers volumineux

betagouv / analyse-flux-insertion

Outil d'analyse des flux et échanges de données dans le domaine de l'insertion

2 stars 1 forks source link

Résolution du bug des fichiers volumineux #33

Closed qblanc closed 3 years ago

qblanc commented 3 years ago

Issue #21
suppression des .map au profit d'une boucle for
Le bug survenait à cause d'énormes arrays de HTMLCollection que l'on constituait pour "folders" et "people". Le stockage de ces arrays dans des variables a été retiré au profit d'une boucle for sur folders. Les opérations se déroulent désormais toutes à l'intérieur, en incrémentant des accum extérieurs à la boucle for.
Issue #27

aminedhobb commented 3 years ago

Petite remarque au passage: Ce serait cool d'expliciter un peu plus ce que tu fais et pourquoi tu le fais dans tes descriptions de PRs. Même si j'ai le contexte, ça peut être utile quand on se replonge dans l'historique ou pour quelqu'un qui rejoindrait le projet. Aussi tu peux lier les issues ici si tu le souhaites (et ça la ferme automatiquement quand la PR est merged):