CesiumGS/cesium

Update zip.js dependency to latest

Fechada

#8.092 aberto em 20 de ago. de 2019

Ver no GitHubGuia do colaborador
 (3 comentários) (0 reação) (0 responsável)JavaScript (3.324 forks)batch import
good first issuetype - enhancement

Métricas do repositório

Stars
 (11.758 estrelas)
Métricas de merge de PR
 (Mesclagem média 15d 12h) (23 fundiu PRs em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
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.
Pilha de tecnologia
javascript
Domain
performance
Tipo Issue
Performance
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Familiarity with npm or yarnUnderstanding of dependency updatesBasic knowledge of Cesium's build process
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
85