tiatoolbox icon indicating copy to clipboard operation
tiatoolbox copied to clipboard

Zarr 3.0 compatibility

Open measty opened this issue 1 year ago • 3 comments

Fixes the error related to zarr.errors.FSPathExistNotDir

It doesnt exist anymore since zarr v3.0.0, which instead just raises a standard FileNotFoundError

rather than checking for different error types in different versions of zarr, i've just removed the check for specific error type in favour of any errors

There are some other changes which we need to deal with from 3.0 also, as it has removed or changed zarr.LRUStoreCache, zarr.DirectoryStore, and zarr.SQLiteStore all of which tiatoolbox uses

We would also need to wait for tifffile to update to be compatible with zarr 3.0

See: https://zarr.readthedocs.io/en/latest/user-guide/v3_migration.html for more details

measty avatar Jan 16 '25 13:01 measty

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.88%. Comparing base (d82df5c) to head (2212eef).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #904      +/-   ##
===========================================
- Coverage    99.90%   99.88%   -0.03%     
===========================================
  Files           70       70              
  Lines         8735     8735              
  Branches      1149     1149              
===========================================
- Hits          8727     8725       -2     
- Misses           3        5       +2     
  Partials         5        5              

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Jan 16 '25 14:01 codecov[bot]

Zarr v3 is not compatible with tifffile https://github.com/cgohlke/tifffile/issues/282 https://github.com/czbiohub-sf/iohub/issues/292

shaneahmed avatar Mar 19 '25 10:03 shaneahmed

Zarr 3 is supported by tifffile. However, zarr 3 only supports Python 3.11+.

shaneahmed avatar May 23 '25 09:05 shaneahmed