mapbox/mapbox-gl-js

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

Open

#10.049 geöffnet am 23. Okt. 2020

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (2.203 Forks)batch import
good first issue

Repository-Metriken

Stars
 (10.532 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Das Problem weist darauf hin, dass die Benchmarks FeatureLevelRemove und SourceLevelRemove in bench/benchmarks/remove paint state.js (Zeilen 88-100 und 102-114) identisch sind, was wahrscheinlich auf einen Copy Paste Fehler zurückzuführen ist. Der SourceLevelRemove Benchmark sollte `map.removeFeatureState({source: 'land'})` verwenden, anstatt über Features zu iterieren. Überprüfen Sie die mapbox gl js API Dokumentation für `removeFeatureState` und erwägen Sie, ob eine Entfernung auf Quell Eigenschaftsebene (z. B. `map.removeFeatureState({source: 'land'}, 'bench')`) angemessener ist. Die Korrektur besteht darin, die SourceLevelRemove Benchmark Funktion so zu ändern, dass die Schleife durch den korrekten API Aufruf ersetzt wird, um sicherzustellen, dass die beabsichtigte Funktionalität getestet wird. Führen Sie nach der Änderung die Benchmarks aus, um die Korrektheit zu überprüfen.
Tech Stack
javascript
Domain
testing
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScriptGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60