-
Notifications
You must be signed in to change notification settings - Fork 59
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
Add support for remote datasets #842
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #842 +/- ##
==========================================
- Coverage 92.11% 92.01% -0.10%
==========================================
Files 48 48
Lines 7429 7468 +39
==========================================
+ Hits 6843 6872 +29
- Misses 586 596 +10
🚀 New features to boost your workflow:
|
These are the diffs from the old PR that got closed because of the many merge conflicts. Let's use this code to shape the new implementations.
|
…eate_upath function
… support for other elements
I added my progress. Local read/write behavior should still work. Remote reading for raster (image, labels) elements is added. The tests succeeded locally and I expanded the remote tests for future work on reading and writing all elements. Next steps are:
Out-of-scope for this PR:
|
Thanks @berombau! I will take over for this 😊 (timeline, working on 2-3 PRs and then I want to work on this next). |
for more information, see https://pre-commit.ci
…or to emulate previous ome_zarr.io.parse_url
This is the PR to close #441.