-
-
Notifications
You must be signed in to change notification settings - Fork 290
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
Downstream repositories, on which we should test zarr/master #668
Comments
For context, we are considering changes like PR ( #612 ), which wouldn't expect to cause issues for downstream, but it would be nice to confirm that somehow. Either by testing here or testing on the downstream repos themselves. |
FWIW, Xarray has recently added a nightly ci build that includes zarr-master (example). |
One idea would be to have a dedicated integration test repo for zarr, xarray, dask, and fsspec that would help cover the many edge cases that arise when these libraries all interact. |
Some downstreams (fsspec/s3fs) are tested directly in zarr's suite. Similarly, some combinations of dask/xarray are already tested in places, including their own repos. Testing a workflow that touches all of the parts is tricky, and unlikely to catch the corner cases that arise in real work. For example, many s3 problems are around real cluster use and auth, which is very hard to test against. |
Good; so we are confident that if something break upstream, a bug will be fixed fast :-) |
I don't have an immediate driver for xarray, but I was certainly considering adding to @constantinpape's zarr_implementations for example different versions & at least a few different cloud backends. If such a test suite appears elsewhere, though, happy to adapt. |
We added Zarr |
Awesome ! |
The text was updated successfully, but these errors were encountered: