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

ENH: Use lzw + predictor 3 to compress SAR bands that can be despeckled #173

Open
remi-braun opened this issue Aug 13, 2024 · 2 comments
Open

ENH: Use lzw + predictor 3 to compress SAR bands that can be despeckled #173

remi-braun opened this issue Aug 13, 2024 · 2 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@remi-braun
Copy link
Member

remi-braun commented Aug 13, 2024 •
edited
Loading

SNAP 9 and before wouldn't handle LZW compression with predictor == 3.

❌ Caused by: javax.imageio.IIOException: Illegal value for Predictor in TIFF file

Test it with SNAP10 and update sar_product.SAR_PREDICTOR to 2 if it's working.
@remi-braun remi-braun added the enhancement New feature or request label Aug 13, 2024
@remi-braun
Copy link
Member Author

remi-braun commented Aug 13, 2024 •
edited
Loading

✔️ SNAP 10 handles predictor 3 with LZW compression

@remi-braun remi-braun changed the title ENH: Use lzw + predictor 2 with SNAP10 to compress SAR bands that can be despeckled ENH: Use lzw + predictor 3 with SNAP10 to compress SAR bands that can be despeckled Aug 13, 2024
remi-braun added a commit that referenced this issue Aug 13, 2024
@remi-braun
OPTIM: Use default (and optimized) predictor in rasters.write if SN…
0202839 
…AP is version 10 or higher #173
@remi-braun remi-braun added the help wanted Extra attention is needed label Sep 4, 2024
@remi-braun
Copy link
Member Author

ROLLBACK: don't use "default (and optimized) predictor in rasters.write if SNAP is version 10 or higher" as it depends not on SNAP but on javax.imageio's version.

The problem is I don't know how to check if javax.imageio can use this predictor 🤔

@remi-braun remi-braun reopened this Sep 4, 2024
@remi-braun remi-braun changed the title ENH: Use lzw + predictor 3 with SNAP10 to compress SAR bands that can be despeckled ENH: Use lzw + predictor 3 to compress SAR bands that can be despeckled Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@remi-braun