feat: avoid data cloning in xSampling

mljs · Nov 24, 2023 · 6c2a8bb · 6c2a8bb
1 parent cf03423
commit 6c2a8bb
Showing 1 changed file with 5 additions and 7 deletions.
diff --git a/src/x/xSampling.ts b/src/x/xSampling.ts
@@ -23,17 +23,15 @@ export function xSampling(
     );
   }
 
-  const clonedArray = array.slice();
-  returnArray.push(clonedArray[0]);
-  clonedArray.shift();
-  const delta = Math.floor(clonedArray.length / (length - 1));
+  returnArray.push(array[0]);
+  const delta = Math.floor((array.length - 1) / (length - 1));
 
   for (
-    let i = delta - 1, j = 0;
-    i < clonedArray.length && j < length - 1;
+    let i = delta, j = 0;
+    i < array.length && j < length - 1;
     i = i + delta, j++
   ) {
-    returnArray.push(clonedArray[i]);
+    returnArray.push(array[i]);
   }
 
   return returnArray;