Skip to content

GeoTiff Band Interleave Construction - #2767

Merged
echeipesh merged 4 commits into
locationtech:masterfrom
echeipesh:feature/write-geotiff-band-interleave
Jul 27, 2018
Merged

echeipesh merged 4 commits into
locationtech:masterfrom
echeipesh:feature/write-geotiff-band-interleave

Conversation

@echeipesh

@echeipesh echeipesh commented Jul 24, 2018 •

Copy link
Copy Markdown
Contributor

Overview

Enables building multiband GeoTiffs with band interleave storage method.

Checklist

  • docs/CHANGELOG.rst updated, if necessary
  • [-] docs guides update, if necessary
  • [-] New user API has useful Scaladoc strings
  • Unit tests added for bug-fix or new feature

@echeipesh
echeipesh force-pushed the feature/write-geotiff-band-interleave branch from c10cafb to fa9cc74 Compare July 24, 2018 21:57

@jbouffard jbouffard left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that this PR looks good. Once the CHANGELOG is updated, I'd say this will be good to merge.

@echeipesh
echeipesh merged commit 9d31897 into locationtech:master Jul 27, 2018
@echeipesh echeipesh added this to the 2.0 milestone Aug 3, 2018
echeipesh added a commit that referenced this pull request Aug 5, 2018
GeoTiffs for COGs can be written with band interleave

(cherry picked from commit 3349bf8)

Use GeoTiffBuilder in GeoTiffMultibandTile constructor from MultibandTile

This handles band interleave in that construction casey

(cherry picked from commit f07792e)

Fix: writing actual geotiff interleave method

(cherry picked from commit fa9cc74)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants