Modify

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#15862 closed task (fixed)

mismatching bounds at french imagery entries

Reported by: Klumbumbus Owned by: Stereo
Priority: normal Milestone:
Component: External imagery source Version:
Keywords: Cc:

Description

The bounds of an imagery entry must match the shape. Could you please review your edits? Thanks!

* Bounds do not match shape (is 41.3136241,-5.3525496,51.1761675,9.6356823, calculated <bounds min-lat='41.27688' min-lon='-6.3' max-lat='51.32937' max-lon='9.8'>): [FR] BANO - http://{switch:a,b,c}.layers.openstreetmap.fr/bano/{zoom}/{x}/{y}.png
* Bounds do not match shape (is 41.3136241,-5.3525496,51.1761675,9.6356823, calculated <bounds min-lat='41.27688' min-lon='-6.3' max-lat='51.32937' max-lon='9.8'>): [FR] Route 500™©® - http://{switch:a,b,c}.tile.openstreetmap.fr/route500/{zoom}/{x}/{y}.png
* Bounds do not match shape (is -21.496511,-178.577276,51.1761675,55.9657599, calculated <bounds min-lat='-21.496511' min-lon='-178.577276' max-lat='51.32937' max-lon='55.9657599'>): [FR] BDOrtho IGN - https://proxy-ign.openstreetmap.fr/bdortho/{zoom}/{x}/{y}.jpg
* Bounds do not match shape (is 41.3136241,-5.3525496,51.1761675,9.6356823, calculated <bounds min-lat='41.27688' min-lon='-6.3' max-lat='51.32937' max-lon='9.8'>): [FR] Cadastre - http://tms.cadastre.openstreetmap.fr/*/tout/{zoom}/{x}/{y}.png
* Bounds do not match shape (is 41.3136241,-5.3525496,51.1761675,9.6356823, calculated <bounds min-lat='41.27688' min-lon='-6.3' max-lat='51.32937' max-lon='9.8'>): [FR] BD Carthage - http://{switch:a,b,c}.tile.openstreetmap.fr/route500hydro/{zoom}/{x}/{y}.png

(from ImageryCompare)

(The plugin imagery-xml-bounds helps at creating the shape and bounds.)

Attachments (0)

Change History (13)

comment:1 by Stereo, 7 years ago

Resolution: fixed
Status: assignedclosed

Thank you! I didn't know about that plugin, I'll remember it if I ever edit those bounds again.

comment:2 by Stereo, 7 years ago

Thank you for catching the remaining one, stoecker.

in reply to:  2 comment:3 by stoecker, 7 years ago

Replying to Stereo:

Thank you for catching the remaining one, stoecker.

We admins see a warning until all issues are solved ;-)

comment:4 by Klumbumbus, 7 years ago

@Stereo: You added some explicit max-zoom attributes. However this is automatically detected correct by JOSM in most cases. If this works then please don't add max-zoom. The reason is that the imagery provider may change the max-zoom in the future and we don't notice and then we may miss the best resolution of the tiles.

comment:5 by Stereo, 7 years ago

Sure, I can remove those. Should I also change the documentation at https://josm.openstreetmap.de/wiki/Maps#Documentation to say that it's now an anti-pattern?

comment:6 by stoecker, 7 years ago

@Klumbumbus:

For WMS. This was for TMS.

comment:7 by Klumbumbus, 7 years ago

My experience is that autodetection of the max zoom level works for TMS too.

comment:8 by stoecker, 7 years ago

Causing lots of useless requests to the TMS. That's not recommended!

comment:9 by Klumbumbus, 7 years ago

OK. And for WMS this is derived from the GetCapabilities?

comment:10 by Klumbumbus, 7 years ago

@Stereo, could you please review these two differences of ELI and JOSM database:

* License URL differs ('https://data.public.lu/en/datasets/lidar-projet-pilote-dun-releve-3d-du-territoire-luxembourgeois/' != 'https://data.public.lu/en/datasets/bd-l-ortho-webservices-wms-et-wmts/'): [LU] geoportail.lu LIDAR prototype Nordstad 2017 - http://{switch:wmts1,wmts2}.geoportail.lu/opendata/wmts/ortho_10cm_proto_lidar/GLOBAL_WEBMERCATOR_4_V3/{zoom}/{x}/{y}.jpeg
* Attribution URL differs ('https://act.public.lu/fr/cartographie/lidar/' != 'https://data.public.lu/en/datasets/bd-l-ortho-webservices-wms-et-wmts/'): [LU] geoportail.lu LIDAR prototype Nordstad 2017 - http://{switch:wmts1,wmts2}.geoportail.lu/opendata/wmts/ortho_10cm_proto_lidar/GLOBAL_WEBMERCATOR_4_V3/{zoom}/{x}/{y}.jpeg

(I think it is wrong in JOSMs database)

in reply to:  9 comment:11 by stoecker, 7 years ago

Replying to Klumbumbus:

OK. And for WMS this is derived from the GetCapabilities?

No. WMS has not really zoom-steps. Request are based on a free sections of the world. Max-Zoom there is a kind of workaround for bad servers.

in reply to:  10 ; comment:12 by Stereo, 7 years ago

Replying to Klumbumbus:

@Stereo, could you please review these two differences of ELI and JOSM database:

Fixed, thank you.

So I should leave the max-zoom then?

in reply to:  12 comment:13 by Klumbumbus, 7 years ago

Replying to Stereo:

So I should leave the max-zoom then?

Yes.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Stereo.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.