Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

externalize processStaticMap -> abaculus -> mapnik #86

Closed
indus opened this issue Nov 16, 2018 · 3 comments
Closed

externalize processStaticMap -> abaculus -> mapnik #86

indus opened this issue Nov 16, 2018 · 3 comments

Comments

@indus
Copy link
Contributor

indus commented Nov 16, 2018

I had the same build issue like #66.
After having a look what mapnik is used for it seems to me that I won't use the "staticMap" functionality and therefor wouldn't even need abaculus and mapnik.
Because of the issues that might arise from including a dependency with a build step, the size of this dependency and the uncertain use of the functionality I would wish that it gets provided as an optional extension or plugin.

@mojodna
Copy link
Owner

mojodna commented Nov 18, 2018

Agreed; I just created lambda-tileserver for a related reason. Would you be willing to put together a PR stripping out functionality that uses Mapnik?

It's straightforward enough that another tile server could be easily assembled that includes tessera and adds the static map generation functionality; see https://github.com/stamen/toner-carto/blob/master/server.js for a server that wraps tessera w/ error reporting capabilities.

@indus
Copy link
Contributor Author

indus commented Nov 18, 2018 via email

@indus indus mentioned this issue Nov 18, 2018
mojodna added a commit that referenced this issue Nov 21, 2018
Tessera is a tile server, not a tile renderer (although it can do that),
so while static map generation is convenient, it's not core
functionality.

Refs #86
@mojodna
Copy link
Owner

mojodna commented Nov 21, 2018

Published as v0.14.0. I added a note to the README recommending people use ~0.13.1 if they need static map functionality. If sufficient changes are made to tessera, we can revisit creating a server / plugin that wraps it, but for now, that seems like a reasonable solution.

@mojodna mojodna closed this as completed Nov 21, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants