Skip to content
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

bump n5, bigdataviewer, bigwarp versions #254

Merged
merged 2 commits into from
Feb 12, 2024

Conversation

bogovicj
Copy link
Contributor

@bogovicj bogovicj commented Feb 1, 2024

  • add n5-zstandard

Copy link
Member

@ctrueden ctrueden left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The n5-blosc and n5-universe melting pot builds have failing tests. @bogovicj Let me know if you need any help. Otherwise, I will wait for new releases of these components to be added to this PR, to bring everything back into harmony.

@bogovicj
Copy link
Contributor Author

bogovicj commented Feb 6, 2024

Update: I believe saalfeldlab/n5#113 will fix the n5-blosc issue.
So far, I'm having trouble reproducing the n5-universe error, but will keep digging..

@bogovicj
Copy link
Contributor Author

bogovicj commented Feb 6, 2024

This successful n5-universe build should be the same as the failing melting pot build. Not sure what's going on. Need to leave this for now, will revisit tomorrow.

@imagesc-bot
Copy link

This pull request has been mentioned on Image.sc Forum. There might be relevant details there:

https://forum.image.sc/t/updated-ngff-support-in-fiji-hdf5-n5-zarr-ome-ngff/91705/43

@imagesc-bot
Copy link

This pull request has been mentioned on Image.sc Forum. There might be relevant details there:

https://forum.image.sc/t/updated-ngff-support-in-fiji-hdf5-n5-zarr-ome-ngff/91705/44

* fixes build issue for n5-blosc
@bogovicj
Copy link
Contributor Author

bogovicj commented Feb 8, 2024

Good news - the build succeeds now - this fixed the n5-blosc issue

Bad news is I'm not sure why the n5-universe build had a test failure last time. Will investigate another day I think

@ctrueden ctrueden merged commit 637920e into scijava:master Feb 12, 2024
1 check passed
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.

3 participants