Skip to content

feat: download scenes - #648

Open
jo-mueller wants to merge 6 commits into
ome:masterfrom
jo-mueller:download-scenes
Open

feat: download scenes#648
jo-mueller wants to merge 6 commits into
ome:masterfrom
jo-mueller:download-scenes

Conversation

@jo-mueller

@jo-mueller jo-mueller commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Fixes #507

Small PR that injects the from_ome_zarr and to_ome_zarr io pathway in the download utility function. This will probably be a good way to go for handling the download of other kinds of ome-zarrs as well.

I added the download function to the tests that are already in the scene tests, but in the long-term it will probably be good to have a suite of test data somewhere so we don't need to generate dummy data over and over again in the tests here.

Merging this could constitute another patch release?

@jo-mueller jo-mueller mentioned this pull request Sep 2, 2026
@codecov

codecov Bot commented Sep 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.15%. Comparing base (1d38546) to head (8fee580).
⚠️ Report is 6 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #648      +/-   ##
==========================================
+ Coverage   87.09%   87.15%   +0.06%     
==========================================
  Files          17       17              
  Lines        2518     2531      +13     
==========================================
+ Hits         2193     2206      +13     
  Misses        325      325              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

download failing for RFC-5 CoordinateSystems group

1 participant