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

(0 commentaire) (0 réaction) (0 personne assignée)JavaScript (2 203 forks)batch import

good first issue

Métriques du dépôt

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

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

Direction de recherche: Le problème identifie que les benchmarks FeatureLevelRemove et SourceLevelRemove dans bench/benchmarks/remove paint state.js (lignes 88-100 et 102-114) sont identiques, ce qui est probablement une erreur de copier coller. Le benchmark SourceLevelRemove doit utiliser `map.removeFeatureState({source: 'land'})` au lieu de boucler sur les entités. Consultez la documentation de l'API mapbox gl js pour `removeFeatureState` et envisagez si une suppression au niveau de la propriété de source (par exemple `map.removeFeatureState({source: 'land'}, 'bench')`) est plus appropriée. La correction consiste à modifier la fonction de benchmark SourceLevelRemove pour remplacer la boucle par l'appel API correct, en veillant à ce qu'elle teste la fonctionnalité prévue. Après avoir effectué la modification, exécutez les benchmarks pour vérifier leur exactitude.
Stack technique: javascript
Domaine: testing
Type d'issue: Bug
Difficulté: 2
Temps estimé: Moins d'une heure
Statut d'activité: Récente
Clarté: Claire
Prérequis: JavaScriptGit
Accessibilité débutant: 60