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.

Sign up for GitHub

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

Jump to bottom

Use in-memory icechunk stores in roundtrip tests #376

Open
TomNicholas opened this issue Jan 10, 2025 · 1 comment · May be fixed by #422
Open

Use in-memory icechunk stores in roundtrip tests #376

TomNicholas opened this issue Jan 10, 2025 · 1 comment · May be fixed by #422
Labels
Icechunk 🧊 Relates to Icechunk library / spec internals testing

Comments

@TomNicholas
Copy link
Member

TomNicholas commented Jan 10, 2025 •
edited
Loading

If we used an icechunk store for all tests that involved round-tripping, we could avoid the use of kerchunk in those tests (in conjunction with #374). This would help us move towards pinning zarr-python v3.
@TomNicholas TomNicholas added Icechunk 🧊 Relates to Icechunk library / spec internals testing labels Jan 10, 2025
@sharkinsspatial sharkinsspatial mentioned this issue Jan 27, 2025
Open
4 tasks
@jsignell
Copy link
Contributor

I can take a crack at this one unless someone is already on it.

@jsignell jsignell mentioned this issue Jan 28, 2025
Closed
5 tasks
@TomNicholas TomNicholas mentioned this issue Jan 28, 2025
Open
This was referenced Jan 28, 2025
Closed
Merged
@maxrjones maxrjones moved this from Todo to In Progress in VirtualiZarr Zarr-Python 3.0 migration Jan 29, 2025
This was referenced Jan 30, 2025
Closed
Merged
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Icechunk 🧊 Relates to Icechunk library / spec internals testing
Projects
VirtualiZarr Zarr-Python 3.0 migration
Status: In Progress
Milestone
No milestone
2 participants
@jsignell @TomNicholas