Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use ome-zarr-py to write to zarr #9

Closed
ziw-liu opened this issue Nov 30, 2022 · 4 comments
Closed

Use ome-zarr-py to write to zarr #9

ziw-liu opened this issue Nov 30, 2022 · 4 comments
Assignees
Labels
bug Something isn't working enhancement New feature or request

Comments

@ziw-liu
Copy link
Collaborator

ziw-liu commented Nov 30, 2022

Currently we use custom writers that try to be compliant with OME-NGFF and OME-HCS specs. This was out of necessity since there was no existing tool at the time. However, maintenance overhead to keep up with the specs is high and the writers now write to outdated versions of these formats.

Now that tools have evolved within the community, we should be able to switch to the official Python implementation of OME-NGFF as our writer to OME-Zarr.

@ziw-liu ziw-liu added bug Something isn't working enhancement New feature or request labels Nov 30, 2022
@ziw-liu ziw-liu added this to the 0.0.1 milestone Nov 30, 2022
@ziw-liu ziw-liu self-assigned this Nov 30, 2022
@ziw-liu
Copy link
Collaborator Author

ziw-liu commented Dec 1, 2022

Related: #3.

@ziw-liu
Copy link
Collaborator Author

ziw-liu commented Dec 13, 2022

The official implementation appears to be still far from feature complete. While the reader may prove useful, the writer's APIs does not meet our needs in fine-grained control of data IO.

@ziw-liu ziw-liu removed this from the 0.0.1 milestone Dec 13, 2022
@JoOkuma
Copy link
Member

JoOkuma commented Dec 13, 2022

They update the spec much more frequently than the python API. I think it will take a very long time for it to be feature complete.

If our writer uses cupy for the pyramid generation it would be great.
I implemented a tiled version for large images in here it should work on 3d with minor tuning.

@ziw-liu
Copy link
Collaborator Author

ziw-liu commented Dec 14, 2022

With that I think we can close this for now and focus on our custom writer.

@ziw-liu ziw-liu closed this as not planned Won't fix, can't repro, duplicate, stale Dec 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants