clean up some TODOs

Signed-off-by: Connor Tsui <connor.tsui20@gmail.com>

Author: connortsui20

vortex-tensor/src/encodings/l2_denorm.rs

@@ -19,9 +19,8 @@ use crate::scalar_fns::l2_denorm::normalize_as_l2_denorm;
 pub struct L2DenormScheme;
 
 impl Scheme for L2DenormScheme {
-    // TODO(connor): FIX THIS!!!
     fn scheme_name(&self) -> &'static str {
-        "vortex.tensor.UNSTABLE.l2_denorm"
+        "vortex.tensor.l2_denorm"
     }
 
     fn matches(&self, canonical: &Canonical) -> bool {

vortex-tensor/src/encodings/turboquant/centroids.rs

@@ -28,7 +28,6 @@ const CONVERGENCE_EPSILON: f64 = 1e-12;
 /// Number of numerical integration points for computing conditional expectations.
 const INTEGRATION_POINTS: usize = 1000;
 
-// TODO(connor): Maybe we should just store an `ArrayRef` here?
 /// Global centroid cache keyed by (dimension, bit_width).
 static CENTROID_CACHE: LazyLock<DashMap<(u32, u8), Buffer<f32>>> = LazyLock::new(DashMap::default);
 

vortex-tensor/src/scalar_fns/inner_product.rs

@@ -522,8 +522,7 @@ impl InnerProduct {
 
         // Gate: u8 codes and f32 centroids.
         if codes_prim.ptype() != PType::U8 {
-            // TODO(connor): support wider code widths (u16, u32). TurboQuant only emits u8
-            // codes today, so this is the only path we need for now.
+            // TODO(connor): Should we support wider codes?
             return Ok(None);
         }
         if values_prim.ptype() != PType::F32 {

vortex-tensor/src/scalar_fns/l2_denorm.rs

@@ -245,9 +245,7 @@ impl ScalarFnVTable for L2Denorm {
 
         let flat = extract_flat_elements(normalized.storage_array(), tensor_flat_size, ctx)?;
 
-        // TODO(connor): Theoretically we could model this as a multiplication between the
-        // normalized array and a `RunEnd(Sequence(0, dimensions), norms)`. But since we have
-        // already canonicalized the array, it is probably not faster to do that.
+        // TODO(connor): Do we want a "broadcast" expression for the List types, or is this fine?
         match_each_float_ptype!(flat.ptype(), |T| {
             let norms = norms.as_slice::<T>();