consbio / mbtileserver

Basic Go server for mbtiles
ISC License
658 stars 104 forks source link

Fix handlers for ArcGIS #116

Closed brendan-ward closed 3 years ago

brendan-ward commented 3 years ago

This picks up the ArcGIS handler changes from #115, including fixes to LODs and tile origin point.

Temporarily (possibly long term) disables layer info at the service root and layers endpoint.

Adds an info endpoint normally provided by ArcGIS server.

coveralls commented 3 years ago

Coverage Status

Coverage remained the same at 79.225% when pulling 25fbeee2609150327a5f331207029313306c0327 on fix_arcgis_handlers into 057db5a2eeae9d27933146fb747db1ffd22e4a2f on master.

brendan-ward commented 3 years ago

@reyemtm could you please confirm if this resolves the tile shift issue? I don't have high resolution imagery served via a different service to compare it to.

I was able to successfully add this to AGOL, but I can't confirm that the print is working without having this out on a public-facing server.

brendan-ward commented 3 years ago

I confirmed that the ArcGIS endpoints now support printing in AGOL, but subject to the same AGOL bug of rendering the wrong zoom levels for low quality. So - at least we're working in AGOL.

What is interesting is that AGOL appears to request the zoom level (according to the request log) and requests the same tiles repeatedly (about 8x per tile).

reyemtm commented 3 years ago

That's very strange, especially since the WMTS endpoints (from the other project) print fine. I'll update this on my on and test it out.

Tested. The shift is fixed. However I still get no print output on my end. Do you have the ability to share your service or map, either publicly or with my AGOL org?

Thanks

On Fri, Aug 6, 2021, 10:47 PM Brendan Ward @.***> wrote:

I confirmed that the ArcGIS endpoints now support printing in AGOL, but subject to the same AGOL bug of rendering the wrong zoom levels for low quality. So - at least we're working in AGOL.

What is interesting is that AGOL appears to request the zoom level (according to the request log) and requests the same tiles repeatedly (about 8x per tile).

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/consbio/mbtileserver/pull/116#issuecomment-894594314, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABQ2HUOXSHBHK4ESMULPZWDT3SNEPANCNFSM5BW3LP4Q . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .