-
-
Notifications
You must be signed in to change notification settings - Fork 686
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
Use zstd compression in NRRD and MetaIO #348
Comments
This seems related to this discussion. It does seem like the NRRD format needs to be updated to specifically include this.
|
@dzenanz FYI: I have been assisting zstd with updating their CMake implementation. The community has been fairly receptive to changes that increase it's use. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Just adding updated information:
Current release is now v1.4.3 |
Two recent experiments I did may be relevant to this thread. These both focus on making Gzip compression faster, which would retain compatibility with the current NRRD format. One could use this acceleration to use a better compression level, getting a little closer to zstd. 1.) Since pigz is based on zlib, it can be recompiled with the accelerated CloudFlare zlib. Here is a simple CMake script that builds pigz with the CloudFlare zlib:
2.) ITK uses znzlib to read/write NIfTI images. My accelerated znzlib demonstrates two methods to accelerate writing gzip images. First, the NIfTI IO can be built using the CloudFlare zlib which is faster than the system zlib. Second, the user can provide an environment variable to request that data is piped to pigz. Only a few lines of code are modified, yielding a dramatic performance increase. I do think the same strategy could be used for the NRRD writing. As my project demonstrates, file writing for uncompressed data and piped pigz are the same, so one can generally use the same existing well tested routines. |
The low hanging fruit is zlib-ng, as pointed out in a VTK forum discussion. |
Also #416 |
This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions. |
[Background discussion]
zstd
's advanced API which supports multithreading has recently entered staging area with intention to be moved into stable API. Now is a great time to make use of it, before ITKv5.0 is released.The text was updated successfully, but these errors were encountered: