fix[vortex-array]: fix offset_within_chunk underflow on patches array

slice_start_idx was missing normalization to absolute coordinates.

Signed-off-by: Alfonso Subiotto Marques <alfonso.subiotto@polarsignals.com>

Author: asubiotto

vortex-array/src/patches.rs

@@ -670,12 +670,15 @@ impl Patches {
         let offset_within_chunk = chunk_offsets
             .as_ref()
             .map(|chunk_offsets| -> VortexResult<usize> {
-                let base_offset = chunk_offsets
+                let absolute_chunk_base = chunk_offsets
                     .scalar_at(0)?
                     .as_primitive()
                     .as_::<usize>()
                     .ok_or_else(|| vortex_err!("chunk offset does not fit in usize"))?;
-                Ok(slice_start_idx - base_offset)
+                let absolute_slice_start = self.chunk_offset_at(0)?
+                    + self.offset_within_chunk.unwrap_or(0)
+                    + slice_start_idx;
+                Ok(absolute_slice_start - absolute_chunk_base)
             })
             .transpose()?;
 
@@ -2151,6 +2154,20 @@ mod test {
         );
     }
 
+    #[test]
+    fn test_nested_slice_with_dropped_first_chunk() {
+        // PATCH_CHUNK_SIZE = 1024, so the two patches land in different chunks.
+        let indices = buffer![0u64, 1024].into_array();
+        let values = buffer![1i32, 2].into_array();
+        let chunk_offsets = buffer![0u64, 1].into_array();
+        let patches = Patches::new(2048, 0, indices, values, Some(chunk_offsets)).unwrap();
+
+        // Drop chunk 0, then re-slice the result.
+        let dropped_first = patches.slice(1024..2048).unwrap().unwrap();
+        let resliced = dropped_first.slice(0..1024).unwrap().unwrap();
+        assert_eq!(resliced.num_patches(), 1);
+    }
+
     #[test]
     fn test_index_larger_than_length() {
         let chunk_offsets = buffer![0u64].into_array();