Skip to content

Conversation

melissalinkert
Copy link
Member

Opening as a draft to show some initial progress, but there are still a few things to do here before this is ready for full review:

  • make compression options work with Zarr v3 (right now only default codec supported)
  • add sharding options (--shard-width, --shard-height, --shard-depth to match chunk options?)
  • add tests that compare v2 against v3 output with various combinations of options
  • write correct HCS metadata for v3

A simple test of bin/bioformats2raw --v3 test.fake test-v3.zarr or similar should work though, and v2 output should be unaffected as indicated by passing tests.

}

void checkMultiscale(Map<String, Object> multiscale, String name) {
assertEquals(getNGFFVersion(), multiscale.get("version"));
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure if this multiscale is zarr v3 but in OME-Zarr v0.5, the version is not under multiscale any more but under "attributes": {"ome": {"version": "0.5", "multiscales": {...}

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point, thanks @will-moore. I'll need to update that in both the conversion and the test then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants