Opened 8 years ago

Closed 7 years ago

#5336 closed defect (fixed) generate_tilemapresource south/west should be west/south

Reported by: mj10777 Owned by: warmerdam
Priority: normal Milestone: 1.11.1
Component: Utilities Version:
Severity: normal Keywords: gdal2tiles


It seems that the original author made a mistake when writing this

At present it is:

<Origin x="%(south).14f" y="%(west).14f"/>

And should be:

<Origin x="%(west).14f" y="%(south).14f"/>

Even if this file is not really used, it should be corrected.

  • sample of correct use:

<Origin x="-180" y="-90" />

Attachments (1) (742 bytes) - added by hamish 7 years ago.
patch to fix x & y swap in

Download all attachments as: .zip

Change History (4)

comment:1 Changed 8 years ago by mj10777

I have just noticed that also the BoundingBox? code was also incorrect:

<BoundingBox? minx="%(south).14f" miny="%(west).14f" maxx="%(north).14f" maxy="%(east).14f"/>

This should also be:

<BoundingBox? minx="%(west).14f" miny="%(south).14f" maxx="%(east).14f" maxy="%(north).14f"/>

The final result looking like this:

<BoundingBox? minx="-180.00000000000000" miny="-85.05113000000000" maxx="180.00000000000000" maxy="85.05113000000000"/>

Changed 7 years ago by hamish

Attachment: added

patch to fix x & y swap in

comment:2 Changed 7 years ago by hamish

Component: defaultUtilities
Keywords: gdal2tiles added
Priority: lownormal
Severity: trivialnormal
Version: unspecifiedsvn-trunk


patch against the latest gdal svn attached, it would be nice to have this one fixed as it causes problems for Geopaparazzi.

see also ticket #4670 "gdal2tiles generates tilemapresource.xml with inconsistent SRS/Coordinates" (epsg 900913 vs 4326: should <SRS> match the coord bounds in BoundingBox? or the native projection of the tile's image data?)

thanks, Hamish

comment:3 Changed 7 years ago by Even Rouault

Milestone: 1.11.1
Resolution: fixed
Status: newclosed
Version: svn-trunk

trunk r27348, branches/1.11 r27349 " fix inverted long/lat in BoundingBox? and Origin elements of tilemapresource.xml (patch by hamish, #5336)"

Note: See TracTickets for help on using tickets.