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

srs representations and a small optimization #2

Closed
springmeyer opened this issue Sep 1, 2011 · 4 comments
Closed

srs representations and a small optimization #2

springmeyer opened this issue Sep 1, 2011 · 4 comments
Assignees

Comments

@springmeyer
Copy link
Owner

For the style in this we moved to using the +init syntax to ensure that srs strings match exactly what is passed in from the url request.

But, if we did the translation between +init=epsg:3857 and the exact mercator proj4 literal (that we choose to use in the style) in the server this would avoid proj4 (inside of mapnik's proj_transform) having to hit its /usr/share/proj/epsg file to get the proj4 definition. This lookup only should happen a couple of times max per request, so it may not be a measurable i/o burden, but could still be avoided as a small optimization.

@springmeyer
Copy link
Owner Author

a cleaner solution could be to work on http://trac.mapnik.org/ticket/839, but at this point I don't think this is as critical as other issues.

@kkaefer
Copy link
Contributor

kkaefer commented Sep 2, 2011

That ticket only solves part of the problem. The problem in the style was that we had slightly different proj4 strings which resulted in proj4 being called and it essentially did a null transform. Mapnik should "normalize" proj4 strings to make sure we're only ever using one single proj4 string for all descriptions of a projection.

@springmeyer
Copy link
Owner Author

yes, totally, the main problem was non matching. But given that we are now matching, this could be a future todo - though lets not worry about this right now... (mostly created this ticket for @ajashton to know the backlog)

@ghost ghost assigned springmeyer Sep 2, 2011
@springmeyer
Copy link
Owner Author

Fixed upstream in Mapnik now by implement 4326 and 3857 natively.

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

No branches or pull requests

2 participants