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.

Sign up for GitHub

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

Jump to bottom

SZIP support #435

Closed
JaWSnl opened this issue Apr 9, 2021 · 0 comments · Fixed by #436
Closed

SZIP support #435

JaWSnl opened this issue Apr 9, 2021 · 0 comments · Fixed by #436
Labels
enhancement

Comments

@JaWSnl
Copy link
Contributor

JaWSnl commented Apr 9, 2021

While GZIP is nice, SZIP is performing better for my use case.

Note however that the SZIP that's provided with HDF5 is (or at the very least used to be) commercially licensed for anything but the most basic use case. See https://support.hdfgroup.org/doc_resource/SZIP/Commercial_szip.html for more details.

However libaec provides an SZIP compatible library and that makes SZIP a very useful addition in my opinion. See https://gitlab.dkrz.de/k202009/libaec for more details.

A pull request will be provided shortly.
@alkino alkino linked a pull request Apr 10, 2021 that will close this issue
Add SZIP support (#435) #436
Merged
@JaWSnl JaWSnl mentioned this issue Apr 10, 2021
Merged
matz-e added a commit that referenced this issue Apr 26, 2021
@JaWSnl @alkino @matz-e
Add SZIP support (#435) (#436)
f2a6b1d 
HDF has default support for SZIP, but commercially licensed.
Using `libaec` SZIP can be used with a permissive license.
This commit allows SZIP to be defined as a property.

Co-authored-by: Jan Willem van Silfhout <[email protected]>
Co-authored-by: Nicolas Cornu <[email protected]>
Co-authored-by: Matthias Wolf <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add SZIP support (#435)
2 participants
@pramodk @JaWSnl