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

GeoTiff CellType Conversion Fix #2794

Merged
merged 1 commit into from Aug 30, 2018

Conversation

Projects
None yet
2 participants
@jbouffard
Contributor

jbouffard commented Aug 28, 2018

Overview

This PR fixes the logic in the CellType conversion for GeoTiffs so that they can convert to the UByte and UShort CellTypes correctly.

Checklist

  • docs/CHANGELOG.rst updated, if necessary
  • Unit tests added for bug-fix or new feature

Notes

I didn't update the CHANGELOG because I wasn't sure what the next version of GT we'd be releasing.

Closes #2789

Resolved #2789 by fixing the GeoTiff CellType conversion logic for UB…
…yte and UShort

Signed-off-by: Jacob Bouffard <jbouffard@azavea.com>

Added unit tests

Signed-off-by: Jacob Bouffard <jbouffard@azavea.com>

@echeipesh echeipesh added the bug label Aug 30, 2018

@echeipesh echeipesh added this to the 2.1 milestone Aug 30, 2018

@echeipesh echeipesh merged commit 8faa0b5 into locationtech:master Aug 30, 2018

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
ip-validation
Details
@echeipesh

This comment has been minimized.

Contributor

echeipesh commented Sep 12, 2018

Back-ported to 2.1 with: 6d2694a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment