mapbox/mapbox-gl-js

FeatureLevelRemove- and SourceLevelRemove-Benchmarks in remove_paint_state.js are exactly the same

Open

#10.049 aberto em 23 de out. de 2020

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)JavaScript (2.203 forks)batch import
good first issue

Métricas do repositório

Stars
 (10.532 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Unless I'm missing something, both benchmarks were already the same when they were added in #7761 by @peterqliu :

https://github.com/mapbox/mapbox-gl-js/blob/ff8e087b6e7cc9fe63eb4809a0895b38029a67e2/bench/benchmarks/remove_paint_state.js#L88-L100 https://github.com/mapbox/mapbox-gl-js/blob/ff8e087b6e7cc9fe63eb4809a0895b38029a67e2/bench/benchmarks/remove_paint_state.js#L102-L114

I haven't used these APIs yet, but I believe the SourceLevelRemove should have used something like this?

// No loop
this.map.removeFeatureState({source: 'land'});

Maybe also source-property level like this?

// No loop
this.map.removeFeatureState({source: 'land'}, 'bench');

Guia do colaborador

Direção de pesquisa
O problema identifica que os benchmarks FeatureLevelRemove e SourceLevelRemove em bench/benchmarks/remove paint state.js (linhas 88-100 e 102-114) são idênticos, o que provavelmente é um erro de copiar e colar. O benchmark SourceLevelRemove deve usar `map.removeFeatureState({source: 'land'})` em vez de iterar sobre os recursos. Revise a documentação da API do mapbox gl js para `removeFeatureState` e considere se uma remoção no nível de propriedade da fonte (por exemplo, `map.removeFeatureState({source: 'land'}, 'bench')`) é mais apropriada. A correção envolve modificar a função de benchmark SourceLevelRemove para substituir o loop pela chamada de API correta, garantindo que ela teste a funcionalidade pretendida. Após fazer a alteração, execute os benchmarks para verificar a correção.
Pilha de tecnologia
javascript
Domain
testing
Tipo Issue
Bug
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.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScriptGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60