Skip to content

Add ZStd compression support for GTiff - #3580

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

pomadchin merged 8 commits into
locationtech:masterfrom
dsamaey:feature/gtiff-zstd-support

Conversation

@dsamaey

@dsamaey dsamaey commented Aug 11, 2025 •

Copy link
Copy Markdown
Contributor

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
Copy Markdown
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
Copy Markdown
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]     }

@pomadchin pomadchin left a comment •

Copy link
Copy Markdown
Member

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
Copy Markdown
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
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