From 67a787a55bfd275560838d8fc81a41a0e5969f8a Mon Sep 17 00:00:00 2001 From: Chris Barnes Date: Wed, 14 Jan 2026 16:45:21 +0000 Subject: [PATCH 1/3] RFC-3: review 1 --- rfc/3/reviews/1/index.md | 38 ++++++++++++++++++++++++++++++++++++++ 1 file changed, 38 insertions(+) create mode 100644 rfc/3/reviews/1/index.md diff --git a/rfc/3/reviews/1/index.md b/rfc/3/reviews/1/index.md new file mode 100644 index 00000000..290b08cb --- /dev/null +++ b/rfc/3/reviews/1/index.md @@ -0,0 +1,38 @@ +# RFC-4: Review 1 + +## Review authors + +Chris Barnes + +## Conflicts of interest + +None. + +## Summary + +Ths RFC is an important step towards opening up OME-Zarr to different types of bioimaging. +The current axes specification is already flexible enough that implementations cannot blindly make assumptions about what a particular dimension index means, +while being restrictive enough to be a bit annoying to write validation for. + +This RFC represents an opportunity to simplify the spec and its implementations, +while serving a broader audience. + +## Significant comments and questions + +N/A + +## Minor comments and questions + +The current specification optimises for particular access patterns in terms of memory locality, +and the RFC already makes good points that these optimisations may not hold for other access patterns. +Because of the Zarr transpose codec, +the conceptual layout of the array may not match its on-disk layout, +making the utility of such optimisations even more hazy. +Lastly, with the introduction of the mapAxis transformation in RFC-5, +viewers may not even be retrieving array subsets which match the conceptual layout in zarr. + +Users should have the freedom to design their array layouts to suit their own data and use cases, which this RFC allows. + +## Recommendation + +Accept. From cb385c9fb8b21a42b7365fee669b72d845a90105 Mon Sep 17 00:00:00 2001 From: Chris Barnes Date: Wed, 14 Jan 2026 16:47:54 +0000 Subject: [PATCH 2/3] fix typo --- rfc/3/reviews/1/index.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/rfc/3/reviews/1/index.md b/rfc/3/reviews/1/index.md index 290b08cb..f1b49a27 100644 --- a/rfc/3/reviews/1/index.md +++ b/rfc/3/reviews/1/index.md @@ -10,7 +10,7 @@ None. ## Summary -Ths RFC is an important step towards opening up OME-Zarr to different types of bioimaging. +This RFC is an important step towards opening up OME-Zarr to different types of bioimaging. The current axes specification is already flexible enough that implementations cannot blindly make assumptions about what a particular dimension index means, while being restrictive enough to be a bit annoying to write validation for. From 522bb64b8cd9a62711626cd24c320f5a2a45169e Mon Sep 17 00:00:00 2001 From: Chris Barnes Date: Wed, 14 Jan 2026 17:15:08 +0000 Subject: [PATCH 3/3] Fix typo --- rfc/3/reviews/1/index.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/rfc/3/reviews/1/index.md b/rfc/3/reviews/1/index.md index f1b49a27..e910e624 100644 --- a/rfc/3/reviews/1/index.md +++ b/rfc/3/reviews/1/index.md @@ -1,4 +1,4 @@ -# RFC-4: Review 1 +# RFC-3: Review 1 ## Review authors