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

Grid bounds bug #2364

Merged
merged 2 commits into from Sep 13, 2017

Conversation

Projects
None yet
2 participants
@mteldridge
Contributor

mteldridge commented Sep 8, 2017

Addresses issue #2330

mteldridge added some commits Aug 14, 2017

Added tests to uncover bug in RasterExtent.gridBoundsFor
Signed-off-by: Matthew Eldridge <meldridge@astraea.io>
Proposed fix to grid-bounds-bug. Round-off can yield results that are…
… outside the RasterExtent's boundaries. When `clamp` is set to true, fully clip to the boundaries.

Signed-off-by: Matthew Eldridge <meldridge@astraea.io>
@echeipesh

This comment has been minimized.

Contributor

echeipesh commented Sep 13, 2017

Thank you for both issue and the PR, ❤️

@echeipesh echeipesh merged commit c9e7330 into locationtech:master Sep 13, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
ip-validation
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment