convert integers indexes to int32 #187

Activity

1. Bisaloo commented on May 21, 2026

   

   Bisaloo

   on May 21, 2026

   More actions

   FWIW, R doesn't have int64 either.

   When reading Zarr data with int64, we internally convert it to int32 if the max doesn't exceed what can be stored int32. Using int32 out of the box where possible would slightly improve performance for us as we wouldn't have to convert them.
2. LucaMarconato commented on May 21, 2026

   

   LucaMarconato

   on May 21, 2026

   Member

   More actions

   Thanks for reporting. We could use this helper function

   spatialdata/src/spatialdata/models/_utils.py

   Line 417 in 06e6981

   def _get_uint_dtype(value: int) -> str:

   Related: in reworking with dtypes we should also relax this unnecessary constraint:

   spatialdata/src/spatialdata/_core/operations/rasterize.py

   Line 737 in 06e6981

   if max_label > max_uint16:

   .