JaneliaSciComp / workstation

Janelia Workstation
Other
23 stars 11 forks source link

Update loading behavior for small Zarr chunk sizes in large samples #35

Closed pedson closed 1 year ago

pedson commented 1 year ago

Addresses an issue with large samples (full brain size) with small chunks (128x128x128 or smaller).

There is one small change to non-OME-Zarr content. BasicTileCache.java was hard-coded to use toString() as the key for the map and for messages. This is an expensive operation for OME-Zarr tiles as compared to other formats due to the content we want presented by other parts of the Workstation general code that use it (e.g., tile loading messages). BasicTileCache is a base class so this was changed to method to get the key that defaults to using toString() so existing implementations remain unchanged. The OME-Zarr implementation uses a different method.

This makes the samples loadable, however a future update will be required to reduce the load time further.