search

napari / napari-tiff

A napari reader plugin for tiff images.
BSD 3-Clause "New" or "Revised" License
8 stars 4 forks source link

Add support for ome pyramid (multi-resolution) files #24

Closed folterj closed 4 months ago

folterj commented 7 months ago

Reverted multi-layer code to the original single layer using channel_axis. Extended ome-tiff testing, with metadata and loading existing files, and testing napari layer/metadata.

GenevieveBuckley commented 7 months ago

Ok, I think we have too many ideas happening in one PR. I love that you have lots of improvements, but I think we're going to have to break this up into smaller chunks for make it easier for me to review and merge things.

Can we separate the pyramid support from the ome-tiff metadata handling?

GenevieveBuckley commented 4 months ago

Thank you so much for your contribution @folterj 🎉 This is a really helpful feature to have, especially as there are so many people out there working with multi-resolution files.

I'd also like to apologize to you for both the delay in getting this merged, and for the number of unrelated reviewer comments on this PR.