Skip to content
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

Fix handlers for ArcGIS #116

Merged
merged 3 commits into from
Oct 12, 2021
Merged

Fix handlers for ArcGIS #116

merged 3 commits into from
Oct 12, 2021

Conversation

brendan-ward
Copy link
Collaborator

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
Copy link

coveralls commented Aug 7, 2021

Coverage Status

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

@brendan-ward
Copy link
Collaborator Author

@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
Copy link
Collaborator Author

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
Copy link

reyemtm commented Aug 7, 2021 via email

@brendan-ward brendan-ward merged commit 3536255 into master Oct 12, 2021
@brendan-ward brendan-ward deleted the fix_arcgis_handlers branch October 12, 2021 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants