Skip to content

Add predictor 2 (integer) and predictor 3 (float) support for writing compressed GTiff files - #3588

Merged
pomadchin merged 5 commits into
locationtech:masterfrom
dsamaey:predictor-support-for-writing-gtiff
Sep 20, 2025
Merged

pomadchin merged 5 commits into
locationtech:masterfrom
dsamaey:predictor-support-for-writing-gtiff

Conversation

@dsamaey

@dsamaey dsamaey commented Sep 14, 2025 •

Copy link
Copy Markdown
Contributor

Overview

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 #3587

@dsamaey

dsamaey commented Sep 14, 2025

Copy link
Copy Markdown
Contributor Author

Unrelated test failed (out of heap memory)

@pomadchin

pomadchin commented Sep 14, 2025 •

Copy link
Copy Markdown
Member

Thank you @dsamaey, it looks great! I also need to take care of #3586, was very much loaded recently but will try to merge these two PRs asap.

@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.

This is awesome, thanks for the contribution!

I did some minor code hygiene cleanups, but the entire logic is untouched.

@pomadchin
pomadchin force-pushed the predictor-support-for-writing-gtiff branch from 210a3d2 to 4375159 Compare September 20, 2025 20:32
@pomadchin pomadchin changed the title Predictor support for writing gtiff Predictor 2 & 3 support for writing compressed GTiff files Sep 20, 2025
@pomadchin pomadchin changed the title Predictor 2 & 3 support for writing compressed GTiff files Add predictor 2 (integer) and predictor 3 (float) support for writing compressed GTiff files Sep 20, 2025
@pomadchin
pomadchin merged commit 95c483e into locationtech:master Sep 20, 2025
6 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.

Predictor 2 & 3 support for writing compressed GTiff files

2 participants