Skip to content

Add ZStd compression support for GTiff#3580

Merged
pomadchin merged 8 commits intolocationtech:masterfrom
dsamaey:feature/gtiff-zstd-support
Aug 16, 2025
Merged

Add ZStd compression support for GTiff#3580
pomadchin merged 8 commits intolocationtech:masterfrom
dsamaey:feature/gtiff-zstd-support

Conversation

@dsamaey
Copy link
Contributor

@dsamaey dsamaey commented Aug 11, 2025

Overview

Add ZStd compression support for GTiff

Checklist

  • ./CHANGELOG.md updated, if necessary. Link to the issue if closed, otherwise the PR.
  • Module Hierarchy updated, if necessary
  • docs guides update, if necessary
  • New user API has useful Scaladoc strings
  • Unit tests added for bug-fix or new feature

Demo

Optional. Screenshots/REPL

Notes

Optional. Ancillary topics, caveats, alternative strategies that didn't work out, anything else.

Closes #3579

@pomadchin
Copy link
Member

That is aweseome! May I ask you to sign ECA please to make Eclipse license checker happy?

@pomadchin pomadchin self-requested a review August 11, 2025 19:33
@pomadchin
Copy link
Member

Ooops

[error] /usr/local/src/raster/src/main/scala/geotrellis/raster/io/geotiff/compression/ZStdCompression.scala:50:5: Unmatched closing brace '}' ignored here
test-jdk21-1  | [error]     }

Copy link
Member

@pomadchin pomadchin left a comment

Choose a reason for hiding this comment

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

Thank you so much for the contribution; I'll try to get it in ASAP! 🎉 🎉 🎉

I'll go through the CQ process because of the new dependency later this week and merge it in

@pomadchin
Copy link
Member

Congratulations with the first contribution 🎉 Will merge it once the CI is green!

@pomadchin pomadchin force-pushed the feature/gtiff-zstd-support branch from c80c7d9 to cb84653 Compare August 16, 2025 16:09
@pomadchin pomadchin merged commit dbaac79 into locationtech:master Aug 16, 2025
14 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ZStd compression support

2 participants