-
Notifications
You must be signed in to change notification settings - Fork 27
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 tileserver to pypi #51
Comments
Yes, we discussed this a little bit before but opted to wait until somebody asked for it, and here we are :) It'll imply that all dependent packages are also on pypi, which from a cursory glance looks to be just tilequeue. The work there should mostly be just writing a readme to describe how that might be used. Longer term it might make more sense to split up the code in tilequeue a little bit more. Right now I see it responsible for 2 things:
Tileserver only needs to depend on the first part, and I'm assuming that the 2nd is more specific to Mapzen and less useful to others. But for now, I think we should just publish our existing structure to pypi and possibly make another issue if we want to split up the repo. Any further thoughts @zerebubuth, @iandees, @nvkelso ? |
Pub now, file an issue to split it up for a rainy day.
|
👍 to pub now, 2.0 can be a split if desired.
What does this consist of? |
Managing the tile processing pipeline we have on running on aws, eg sqs, redis, s3. |
Everything except tilequeue is now being pulled from pypi, so it's just it left. Should I open an issue on its repo? |
I've added tilezen/tilequeue#220 – but we also have Dockerization in the mix via #93 so a few different planes in the air right now. |
Anything I can do to help on this? |
@rmarianski can you take a fresh look at this, please? |
Apologies for this taking so long. I'll go ahead and push over the latest vector-datasource, tileserver, and tilequeue releases to pypi tomorrow morning. |
|
Is it possible to add tileserver to pypi? I could take this on, but would much rather a maintainer does so.
The text was updated successfully, but these errors were encountered: