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

Add 'scale factor' option to Mapnik image provider #157

Merged
merged 2 commits into from Oct 13, 2014

Conversation

jwass
Copy link
Contributor

@jwass jwass commented Sep 11, 2013

Adds the optional 'scale factor' argument in the config for Mapnik's
image provider. Allows a TileStache layer to be configured for retina.

This allows specifying the 'scale_factor' option the ImageProvider. It's used to render retina tiles.

For discussions about it, see:
https://github.com/mapnik/mapnik/wiki/Scale-factor
http://support.mapbox.com/discussions/mapbox-ios-sdk/275-retina-maps-for-completely-custom-style
https://groups.google.com/forum/#!topic/mapnik/g4MiIZu9SCk

jwass added 2 commits May 29, 2014 20:50
Adds the optional 'scale factor' argument in the config for Mapnik's
image provider. Allows a TileStache layer to be configured for retina.
Remove improper underscore
@brianreavis
Copy link

@jwass Aside: just started using this and it works like a charm – thanks!

@jwass
Copy link
Contributor Author

jwass commented Oct 13, 2014

@brianreavis Glad to hear it!

@rburhum: I can rebase this on current master, but will be difficult to add to the tests... let me know if I'm wrong about that. Anything else I can do to get this merged?

rburhum added a commit that referenced this pull request Oct 13, 2014
Add 'scale factor' option to Mapnik image provider
@rburhum rburhum merged commit c620165 into TileStache:master Oct 13, 2014
@rburhum
Copy link
Contributor

rburhum commented Oct 13, 2014

I looked at the change. Looks good. thanks @jwass !

@jwass
Copy link
Contributor Author

jwass commented Oct 16, 2014

Thanks!

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.

None yet

3 participants