restict ngff-zarr to prevent errors in Ubuntu 18 #89
Conversation
This file contains 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
kmitcham
requested review from
toloudis,
hughes036,
evamaxfield,
aswallace,
SeanLeRoy,
yrkim98 and
BrianWhitneyAI
evamaxfield
reviewed
Feb 4, 2025
There was a problem hiding this comment.
I am fine with this but I believe #80 will also fix this problem
toloudis
approved these changes
Feb 4, 2025
BrianWhitneyAI
approved these changes
Feb 4, 2025
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.
(resolved before issue created)
Chantelle was encountering issues with bioio writer imports when using latest versions of bio.
We eventually traced the issue to Ubuntu 18 having issues with ngff-zarr>0.9.0, specifically its dependencies in
itkwasm and wasmtime. I don't think I need to pin the wasm dependencies, but if we do, the versions are
itkwasm==1.0b171 wasmtime==22
pip install ngff-zarr==0.9.0 itkwasm==1.0b171 wasmtime==22
Description of Changes
Put a maximum version number on ngff-zarr to avoid low level lib errors.