CesiumGS/cesium

Update zip.js dependency to latest

Chiusa

#8092 aperta il 20 ago 2019

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (0 assegnatari)JavaScript (3324 fork)batch import
good first issuetype - enhancement

Metriche repository

Star
 (11.758 stelle)
Metriche merge PR
 (Merge medio 15g 12h) (23 PR mergiate in 30 g)

Descrizione

Per the zip.js devs, Cesium is using an older build and it results in instantiating a new web worker for every file added to a ZIP (!!!), greatly slowing the process. This only showed up recently because the KML exporter is probably the first part of Cesium that needs to make ZIP files.

Guida contributor

Direzione di ricerca
Investigate the current zip.js version used in Cesium by searching package.json or equivalent files. Refer to zip.js issue #139 (https://github.com/gildas lormeau/zip.js/issues/139) for the recommended version that fixes the excessive worker instantiation. Update the dependency version in the appropriate configuration file and verify that the KML exporter works correctly without performance degradation.
Tech stack
javascript
Dominio
performance
Tipo issue
Performance
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Familiarity with npm or yarnUnderstanding of dependency updatesBasic knowledge of Cesium's build process
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85