Skip to content

Tkakar/add wrapper for precomputed - #509

Merged
tkakar merged 10 commits into
mainfrom
tkakar/add-wrapper-for-precomputed
Jul 30, 2026
Merged

Tkakar/add wrapper for precomputed#509
tkakar merged 10 commits into
mainfrom
tkakar/add-wrapper-for-precomputed

Conversation

@tkakar

@tkakar tkakar commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

This PR adds the wrappers for consuming precomputed data (segmentations and annotations) for neuroglancer

@keller-mark

Copy link
Copy Markdown
Member

It seems like the tests are failing due to changes in ome-zarr package. The latest version of the pypi package 0.18.0 is being installed. The last time the github actions ran on the main branch shows that the version of ome-zarr was 0.15.0.

The short-term fix for this PR would be to pin the version of ome-zarr to 0.15.0, but in the longer term we should update our usage in vitessce-python to be compatible with the latest version

@tkakar

tkakar commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator Author

@keller-mark thanks for the comment, it needed to pin spatialData as well since the 0.18 version needed updated dependencies as well.

@tkakar
tkakar requested a review from keller-mark July 30, 2026 18:50
@tkakar
tkakar merged commit 0296da9 into main Jul 30, 2026
14 checks passed
@tkakar
tkakar deleted the tkakar/add-wrapper-for-precomputed branch July 30, 2026 23:25
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.

2 participants