mapbox/mapbox-gl-js

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

Open

#10.049 aperta il 23 ott 2020

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (2203 fork)batch import
good first issue

Metriche repository

Star
 (10.532 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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');

Guida contributor

Direzione di ricerca
Il problema identifica che i benchmark FeatureLevelRemove e SourceLevelRemove in bench/benchmarks/remove paint state.js (righe 88-100 e 102-114) sono identici, il che probabilmente è un errore di copia incolla. Il benchmark SourceLevelRemove dovrebbe utilizzare `map.removeFeatureState({source: 'land'})` invece di iterare sulle feature. Rivedere la documentazione API di mapbox gl js per `removeFeatureState` e considerare se una rimozione a livello di proprietà della sorgente (ad esempio `map.removeFeatureState({source: 'land'}, 'bench')`) sia più appropriata. La correzione prevede di modificare la funzione di benchmark SourceLevelRemove per sostituire il ciclo con la chiamata API corretta, garantendo che testi la funzionalità prevista. Dopo aver apportato la modifica, eseguire i benchmark per verificarne la correttezza.
Tech stack
javascript
Dominio
testing
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60