cleanup

Signed-off-by: Andrew Duffy <andrew@a10y.dev>

Author: a10y

vortex-cuda/benches/alp_cuda.rs

@@ -26,6 +26,7 @@ use vortex::array::validity::Validity::NonNullable;
 use vortex::buffer::Buffer;
 use vortex::dtype::NativePType;
 use vortex::encodings::alp::ALPArray;
+use vortex::encodings::alp::ALPArrayExt;
 use vortex::encodings::alp::ALPFloat;
 use vortex::encodings::alp::alp_encode;
 use vortex::error::VortexExpect;
@@ -58,7 +59,7 @@ where
     };
     let outlier = T::from(std::f64::consts::PI).unwrap();
 
-    let values: Vec<T> = (0..len)
+    let values: Buffer<T> = (0..len)
         .map(|i| {
             if patch_interval != usize::MAX && i % patch_interval == 0 {
                 outlier
@@ -68,13 +69,22 @@ where
         })
         .collect();
 
-    let primitive_array = PrimitiveArray::new(Buffer::from(values), NonNullable);
-    alp_encode(
+    let primitive_array = PrimitiveArray::new(values, NonNullable);
+    let encoded = alp_encode(
         primitive_array.as_view(),
         None,
         &mut LEGACY_SESSION.create_execution_ctx(),
     )
-    .vortex_expect("failed to ALP-encode array")
+    .vortex_expect("failed to ALP-encode array");
+
+    if patch_frequency > 0.0 {
+        assert!(
+            encoded.patches().is_some(),
+            "expected patches for patch_frequency={patch_frequency}",
+        );
+    }
+
+    encoded
 }
 
 fn benchmark_alp_decode_typed<T>(c: &mut Criterion, type_name: &str)