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

[GEOS-8257] Correct Y order for geopackage WPS tiles output. #2573

Merged
merged 1 commit into from Feb 6, 2018

Conversation

Projects
None yet
3 participants
@jodygarnett
Member

jodygarnett commented Oct 6, 2017

[GEOS-8257] Correct Y order for geopackage WPS tiles output.
GeoPackage first tile is top left (flipped from mbtiles). This is already handled for
Map output, just incorrect for WPS tiles.

Signed-off-by: Jody Garnett <jody.garnett@gmail.com>

# Conflicts:
#	src/community/geopkg/src/main/java/org/geoserver/geopkg/wps/GeoPackageProcess.java
formatOptions.put("format",tiles.getFormat());
formatOptions.put("flipy", "true");
if (tiles.getFormat() != null) {
formatOptions.put("format",tiles.getFormat());

This comment has been minimized.

@jodygarnett

jodygarnett Oct 6, 2017

Member

Had to resolve a small conflict here, @bradh can you review pls

@jodygarnett

jodygarnett Oct 6, 2017

Member

Had to resolve a small conflict here, @bradh can you review pls

@jodygarnett jodygarnett requested a review from bradh Oct 6, 2017

@bradh

bradh approved these changes Oct 6, 2017

@jodygarnett

This comment has been minimized.

Show comment
Hide comment
@jodygarnett

jodygarnett Dec 12, 2017

Member

@bradh running travis again to be sure, I know it has been a while since you reviewed.

Member

jodygarnett commented Dec 12, 2017

@bradh running travis again to be sure, I know it has been a while since you reviewed.

@bencaradocdavies

This comment has been minimized.

Show comment
Hide comment
@bencaradocdavies

bencaradocdavies Feb 6, 2018

Member

Close-reopen to retest on Travis.

Member

bencaradocdavies commented Feb 6, 2018

Close-reopen to retest on Travis.

@bencaradocdavies bencaradocdavies merged commit d5c5604 into geoserver:2.12.x Feb 6, 2018

1 check passed

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