try proj4leaflet to support projection in /map viewer
#560
Conversation
| '{{ tms.crs.to_proj4() }}', { | ||
| origin: [{{ tms.xy_bbox[0] }}, {{ tms.xy_bbox[3] }}], | ||
| // bounds: [[{{ tms.xy_bbox[0] }}, {{ tms.xy_bbox[1] }}], [{{ tms.xy_bbox[2] }}, {{ tms.xy_bbox[3] }}]], | ||
| resolutions: {{ resolutions|safe }}, |
There was a problem hiding this comment.
It seems to work but it needs more testing
There was a problem hiding this comment.
I'm not sure if resolutions is https://github.com/developmentseed/morecantile/blob/main/morecantile/models.py#L370-L379
There was a problem hiding this comment.
So this is a bit tricky. I'll quote myself from https://gis.stackexchange.com/a/423647/53755 :
The result of
L.CRS.scale()has to be multiplied by a CRS-dependant constant (the world bounds according to that CRS) in order to make the numbers meaningful, which IMO leads to confusion.
So you have to be careful about the CRS's bounds, specially if you plan on using TMS-style tiles with an inverted Y-coordinate (a {-y} in the URL template). Specifically, see https://stackoverflow.com/a/62625435/4768502
See also https://gis.stackexchange.com/a/310124/53755 - Don't be confused when using Leaflet's functions for projecting/unprojecting coordinates because they don't rely on CRS-relative numbers, but rather screen-relative numbers.
There was a problem hiding this comment.
Thanks for you help @IvanSanchez
going back at this today, and I'm not quite sure I understand all of ☝️ 😭
vincentsarago
added
enhancement
vincentsarago
changed the title
/map viewer
…nto mapViewerLeafletTMS
| data.tiles[0], { | ||
| minZoom: data.minzoom, | ||
| maxZoom: data.maxzoom, | ||
| bounds: L.latLngBounds([bottom, left], [top, right]), |
There was a problem hiding this comment.
I don't think you want to do this - the bounds option on a L.Tilelayer is a constraint, so the tilelayer will load less tiles. It's not an informative field.
Focus on getting the bounds of the Leaflet CRS right - tilelayers will work throughout that range, and only throughout that range.
|
🥳 adding the markers really help 🙏 @IvanSanchez I can now see which TMS are 🙅 👇 |
|
Alright everything seems to work well 🥳 notes:
|
closes #559