-
Notifications
You must be signed in to change notification settings - Fork 16
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
Zarr.jl #42
Comments
Yes, of course. I always wanted to do this but did not get to it yet. I have a few unit tests that compare against zarr-python (https://github.com/meggart/Zarr.jl/blob/master/test/python.jl), but something more systematic would be great. |
A perhaps interesting test to include: JuliaIO/Zarr.jl#78 (comment) (see also: zarr-developers/community#41) |
I'll volunteer to help on this (or perhaps I have been volunteered) although I think @meggart has the lead here. What needs to be done? Maybe the first step is to address JuliaIO/Zarr.jl#25. I'm already a member of https://github.com/JuliaIO, so I'm biased in that direction. |
Thanks for the offer, @mkitti! I've updated the description with the steps that I know of. From the perspective of this repo, happy to point to whatever org even temporarily and we can always update when JuliaIO/Zarr.jl#25 is settled. |
I agree with @joshmoore . Thanks a lot for offering your help here and we can start writing these coparison tests for Julia independent on where the package lives. Thanks also for updating the todo list @mkitti let me know what kind of support you would need to write the tests, of course I would be available for reviewing code. |
The instructions in the docs proposed in #44 should help. Please provide feedback there if you find things that should be added or clarified. |
Zarr.jl is now in JuliaIO. |
See: https://github.com/meggart/Zarr.jl
@meggart, would there be interest in having Zarr.jl tested against other implementations?
Steps:
The text was updated successfully, but these errors were encountered: