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

nii.zst support #4443

Open
chrisadamsonmcri opened this issue Feb 2, 2024 · 5 comments
Open

nii.zst support #4443

chrisadamsonmcri opened this issue Feb 2, 2024 · 5 comments
Labels
type:Enhancement Improvement of existing methods or implementation

Comments

@chrisadamsonmcri
Copy link

Description

Add Zstd support for NIFTI files so that .nii.zst files can be read and written. Similar to issue #348. This would be great for compression/decompression speed and the file sizes are slightly smaller than those produced by Deflate.

@chrisadamsonmcri chrisadamsonmcri added the type:Enhancement Improvement of existing methods or implementation label Feb 2, 2024
@dzenanz
Copy link
Member

dzenanz commented Feb 2, 2024

This might not be too hard. But this is the first time I hear about .nii.zst 😄

Anyway, it would be good if someone had the time to take this up.

@thewtex
Copy link
Member

thewtex commented Feb 2, 2024

Has .zst support been added to the NIFTI standard?

BTW, there is .iwi.cbor.zst support in ITK-Wasm.

@gdevenyi
Copy link
Contributor

gdevenyi commented Feb 2, 2024

Is a wrapping extension really anything to do with the standard? I would argue its a compression helper.

This is not like minc with built in blocked hdf5 compression, you need to uncompress the whole file to do anything with it.

@thewtex
Copy link
Member

thewtex commented Feb 4, 2024

Most applications support .nii.gz, and if we want data created by one application to be used directly in another, then it is helpful to have its use standardized. Ideally explicitly, but at least as a standard by convention. .nii.gz is common and supported by ITK, nibabel, and others, but .nii.bz2, .nii.xz, .nii.zip are not. If the others were distributed, then to use them they have to be explicitly decompressed outside the application.

@chrisadamsonmcri
Copy link
Author

I understand that it is not widely supported. But that is why I spammed other software boards with this suggestion :) For it to be useful, you are right, it would need to de facto standardized but I guess you have to start somewhere.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:Enhancement Improvement of existing methods or implementation
Projects
None yet
Development

No branches or pull requests

4 participants