[ENH] Add OME-Zarr as a supported format for imaging data#2392
[ENH] Add OME-Zarr as a supported format for imaging data#2392kabilar wants to merge 17 commits into
Conversation
|
I fixed most of the CI failures. The remaining 2 failures appear unrelated. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #2392 +/- ##
=======================================
Coverage 83.07% 83.07%
=======================================
Files 22 22
Lines 1696 1696
=======================================
Hits 1409 1409
Misses 287 287 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
effigies
left a comment
There was a problem hiding this comment.
I support this. It is in line with the decision made at the Copenhagen 2023 meeting which was: HDF5 and Zarr are preferred formats for new multi-dimensional data.
It looks like we dropped the ball on the wording in #1614, but I do think the decision should hold, nonetheless.
|
That's shaping up nicely! Please complement with a PR against bids-examples which would demo some fake dataset with .ome.zarr . FWIW -- here I am creating one but scripting so I could easily recreate etc |
Thanks @yarikoptic. There was already a |
|
Hi @yarikoptic, just following up on my recent commits to see if you have any feedback, thanks. |
yarikoptic
left a comment
There was a problem hiding this comment.
From what I see -- looks good to me
Co-authored-by: Yaroslav Halchenko <debian@onerussian.com>
|
@effigies - we have 2 approvals. This is a tiny PR with HUGE potential consequences. I would love us to proceed on it regardless since I feel it is the right thing to do! |
Fix #1704
cc @satra @balbasty @ayendiki