NIFTI-Imaging / nifti_clib

C libraries for NIFTI support
Other
33 stars 24 forks source link

Keep an eye on zlib-ng #32

Open gdevenyi opened 5 years ago

gdevenyi commented 5 years ago

Good things are happening in the zlib world: https://github.com/zlib-ng/zlib-ng

hjmjohnson commented 5 years ago

Also consider: https://github.com/facebook/zstd

hjmjohnson commented 5 years ago

See conversations related to this topic at:

This is a upstream ITK question, of which there's already some discussion: InsightSoftwareConsortium/ITK#348

Also, https://github.com/zlib-ng/zlib-ng is trying to drag the "official" zlib implementation forward in a non-breaking way.

https://github.com/ANTsX/ANTs/issues/702#issuecomment-454792395

hjmjohnson commented 3 years ago

@gdevenyi It seems that this topic is gaining traction again. Do you have any recommendations?

gdevenyi commented 3 years ago

I'm currently implementing zlib-ng in ITK and also looking at updating Niftilib in ITK. That may involve me working on this.

dzenanz commented 3 years ago

zlib-ng in ITK is ready for a spin. It also increases compression speed for .nii.gz files.

gdevenyi commented 3 years ago

We shall continue to keep an eye on zlib-ng with the hopes of minimizing our divergence from the main codebase (when they merge the prefix code)

dzenanz commented 1 year ago

Prefix code is part of zlib-ng. Time to give this another look?