Add JPEG 2000 codec with optional nvJPEG2000 GPU acceleration#1049
Open
brendancol wants to merge 4 commits intogeotiff-reader-writerfrom
Open
Add JPEG 2000 codec with optional nvJPEG2000 GPU acceleration#1049brendancol wants to merge 4 commits intogeotiff-reader-writerfrom
brendancol wants to merge 4 commits intogeotiff-reader-writerfrom
Conversation
CPU path via glymur (same pattern as JPEG/Pillow and ZSTD/zstandard). GPU path via nvJPEG2000 ctypes bindings (same pattern as nvCOMP). Both are optional -- graceful fallback if libraries aren't installed.
- 14 new tests: codec roundtrip, TIFF write/read roundtrip, public API, availability checks, and ImportError fallback - Fix jpeg2000_compress: calculate numres from tile dimensions, remove pre-existing temp file before glymur write - Update test_edge_cases: use 'webp' as unsupported compression example since 'jpeg2000' is now supported
Covers write/read roundtrip, file size comparison with deflate, multi-band RGB example, and GPU acceleration notes.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #1048
Summary
glymur(optional, same pattern as Pillow/JPEG and zstandard/ZSTD)Test plan
_writer.write()and publicwrite_geotiff/read_geotiff