comments

spiraldb · Jun 6, 2024 · cc1f3de · cc1f3de
1 parent 89ec624
commit cc1f3de
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 25 deletions.
diff --git a/bench-vortex/src/public_bi_data.rs b/bench-vortex/src/public_bi_data.rs
@@ -503,7 +503,6 @@ impl BenchmarkDataset for BenchmarkDatasets {
         self.as_uncompressed();
         for f in self.list_files(FileType::Csv) {
             info!("Compressing {} to lance", f.to_str().unwrap());
-            println!("Compressing {} to lance", f.to_str().unwrap());
             let output_fname = f
                 .file_name()
                 .unwrap()

diff --git a/vortex-alp/src/compute.rs b/vortex-alp/src/compute.rs
@@ -88,29 +88,10 @@ mod test {
 
         let flat = alp.as_contiguous(&[encoded]).unwrap();
 
-        let a = scalar_at(&flat, 0)
-            .unwrap()
-            .value()
-            .as_pvalue()
-            .unwrap()
-            .unwrap();
-        let a: f64 = a.try_into().unwrap();
-
-        let b = scalar_at(&flat, 1)
-            .unwrap()
-            .value()
-            .as_pvalue()
-            .unwrap()
-            .unwrap();
-        let b: f64 = b.try_into().unwrap();
-
-        let c = scalar_at(&flat, 2)
-            .unwrap()
-            .value()
-            .as_pvalue()
-            .unwrap()
-            .unwrap();
-        let c: f64 = c.try_into().unwrap();
+        let a: f64 = scalar_at(&flat, 0).unwrap().try_into().unwrap();
+        let b: f64 = scalar_at(&flat, 1).unwrap().try_into().unwrap();
+
+        let c: f64 = scalar_at(&flat, 2).unwrap().try_into().unwrap();
 
         assert_eq!(a, 1.0);
         assert_eq!(b, 2.0);

diff --git a/vortex-array/src/compute/as_contiguous.rs b/vortex-array/src/compute/as_contiguous.rs
@@ -23,7 +23,7 @@ macro_rules! impl_default_as_contiguous_fn {
                     vortex_error::vortex_bail!(ComputeError: "mismatched dtypes in call to as_contiguous");
                 }
 
-                let mut chunks = Vec::with_capacity(arrays.iter().map(|array| array.len()).sum());
+                let mut chunks = Vec::with_capacity(arrays.len());
                 for array in arrays {
                     chunks.push(array.clone().flatten()?.into_array());
                 }