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.

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

🎨 Initial CogReader struct with decoder field #7

Merged
merged 1 commit into from
Mar 10, 2024
Merged

Conversation

weiji14
Copy link
Owner

@weiji14 weiji14 commented Mar 10, 2024

Defining a Cloud-Optimized GeoTIFF Reader struct with a decoder field containing an instance of tiff::decoder::Decoder with an unlimited decoding buffer size by default.

Note that this struct is marked as private for now until more method implementations are done. Still debating on what are considered core fields/attributes for GeoTIFFs (affine transformation?), and what are 'optional'.

And yes, this struct should ideally be upstreamed somehow to https://github.com/georust/geotiff or somewhere, but I'm gonna experiment with different APIs over the next few weeks first.

References:

Defining a Cloud-Optimized GeoTIFF Reader struct with a decoder field containing an instance of tiff::decoder::Decoder with an unlimited decoding buffer size by default. Marking the struct as private for now until more method implementations are done.
@weiji14 weiji14 added the feature New feature or request label Mar 10, 2024
@weiji14 weiji14 added this to the 0.1.0 milestone Mar 10, 2024
@weiji14 weiji14 self-assigned this Mar 10, 2024
@weiji14 weiji14 marked this pull request as ready for review March 10, 2024 03:56
@weiji14 weiji14 enabled auto-merge (squash) March 10, 2024 03:56
@weiji14 weiji14 merged commit 2e463a7 into main Mar 10, 2024
13 checks passed
@weiji14 weiji14 deleted the struct/CogReader branch March 10, 2024 04:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant