fix(pointcloud): remove unused variable

This commit fixes a left-over from #719
iTowns · Apr 16, 2018 · e0d839d · e0d839d
1 parent 19ae01c
commit e0d839d
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 14 deletions.
diff --git a/examples/pointcloud.js b/examples/pointcloud.js
@@ -77,9 +77,7 @@ function showPointcloud(serverUrl, fileName, lopocsTable) {
             var info = document.getElementById('info');
             oldPostUpdate.apply(pointcloud, arguments);
             info.textContent = 'Nb points: ' +
-                pointcloud.counters.displayedCount.toLocaleString() + ' (' +
-                Math.floor(100 * pointcloud.counters.displayedCount / pointcloud.counters.pointCount) + '%) (' +
-                view.mainLoop.gfxEngine.renderer.info.memory.geometries + ')';
+                pointcloud.displayedCount.toLocaleString();
         };
         window.view = view;
     }

diff --git a/examples/pointcloud_globe.js b/examples/pointcloud_globe.js
@@ -50,9 +50,7 @@ function showPointcloud(serverUrl, fileName) {
             var info = document.getElementById('info');
             oldPostUpdate.apply(pointcloud, arguments);
             info.textContent = 'Nb points: ' +
-                pointcloud.counters.displayedCount.toLocaleString() + ' (' +
-                Math.floor(100 * pointcloud.counters.displayedCount / pointcloud.counters.pointCount) + '%) (' +
-                view.mainLoop.gfxEngine.renderer.info.memory.geometries + ')';
+                pointcloud.displayedCount.toLocaleString();
         };
         window.view = view;
     }

diff --git a/src/Process/PointCloudProcessing.js b/src/Process/PointCloudProcessing.js
@@ -291,22 +291,20 @@ export default {
             return;
         }
 
-        layer.counters = {
-            displayedCount: 0,
-        };
+        layer.displayedCount = 0;
         for (const pts of layer.group.children) {
-            layer.counters.displayedCount += pts.geometry.drawRange.count;
+            layer.displayedCount += pts.geometry.drawRange.count;
         }
 
-        if (layer.counters.displayedCount > layer.pointBudget) {
-            const reduction = layer.pointBudget / layer.counters.displayedCount;
+        if (layer.displayedCount > layer.pointBudget) {
+            const reduction = layer.pointBudget / layer.displayedCount;
             for (const pts of layer.group.children) {
                 if (pts.material.visible) {
                     const count = Math.max(1.0, Math.floor(pts.geometry.drawRange.count * reduction));
                     pts.geometry.setDrawRange(0, count);
                 }
             }
-            layer.counters.displayedCount *= reduction;
+            layer.displayedCount *= reduction;
         }
 
         const now = Date.now();

diff --git a/utils/debug/PointCloudDebug.js b/utils/debug/PointCloudDebug.js
@@ -153,7 +153,7 @@ export default {
                                 elt.obj.material.visible = layer.dbgDisplayParents;
 
                                 if (v && !elt.obj.material.visible) {
-                                    layer.counters.displayedCount -= elt.obj.geometry.drawRange.count;
+                                    layer.displayedCount -= elt.obj.geometry.drawRange.count;
                                 }
                             }
                         }