add remote support #971
Draft
add remote support #971
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
for more information, see https://pre-commit.ci
This reverts commit 1f8a01c.
Co-authored-by: LucaMarconato <[email protected]>
…eate_upath function
… support for other elements
for more information, see https://pre-commit.ci
…or to emulate previous ome_zarr.io.parse_url
…eate_upath function
… support for other elements
for more information, see https://pre-commit.ci
…or to emulate previous ome_zarr.io.parse_url
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #971 +/- ##
==========================================
+ Coverage 92.11% 92.39% +0.28%
==========================================
Files 48 48
Lines 7429 7453 +24
==========================================
+ Hits 6843 6886 +43
+ Misses 586 567 -19
🚀 New features to boost your workflow:
|
* tests: update tests to use remote paths * spatialdata._io._utils: support consolidated remote stores * spatialdata._io.format: supprot remote embl datasets on s3 --------- Co-authored-by: Wouter-Michiel Vierdag <[email protected]>
Co-authored-by: Wouter-Michiel Vierdag <[email protected]>
Co-authored-by: Wouter-Michiel Vierdag <[email protected]>
Co-authored-by: Wouter-Michiel Vierdag <[email protected]>
* refactor StoreLike to types * support remote write points and shapes * cleanup * add additional remote write support * update dependency ome-zarr * revert update dependency * support remote write tables
|
@ap-- @berombau been a long time coming but almost ready to get all the work in regarding remote read and write support. Will add some tests for azure still as well. One blocker currently is that we require the newest version of ome-zarr which includes a fix that I implemented for writing images and labels on remote. The problem is that this version is only available with zarr>3. |
|
Awesome! And thanks for all the work 🙏 Great to see that this will be included. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.