Skip to content

Conversation

@dsamaey
Copy link
Contributor

@dsamaey dsamaey commented Sep 14, 2025

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
Copy link
Contributor Author

dsamaey commented Sep 14, 2025

Unrelated test failed (out of heap memory)

@pomadchin
Copy link
Member

pomadchin commented Sep 14, 2025

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.

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.

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