-
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
ImportError: cannot import name DataFetcher in v2.1.0, wrong tilequeue version in requirements #115
Comments
This would also be an issue if someone followed the current version of the vector-datasource install guide using the checkout versions listed there. |
Yea, good point, those do have master set for those releases. Those instructions are usually tested just with master, and I take the steps you describe above when switching versions. We should probably at a minimum update the documentation to call this out, and ensure that going forward releases have specific versions set in their requirements files. Instead of having requirements files in each repo though, I think we should have a single set of requirements just in vector-datasource that assumes installation of everything for setup purposes, since that's the useful part practically anyway. We could either split apart just the upstream dependencies from the versions of the tilezen repos, and make separate explicit pip install steps for the install or tilezen repos which might make this more visible. That would also mean that we wouldn't have the problem of having to update the requirements files when we release. Or we just add a step to the instructions that says to edit the bottom of the requirements file for the particular versions desired. What does everyone think? |
So Then the instructions to get a "standard install" would be to clone |
Thanks @radumas for posting this fix. I was stuck after following the install guide. |
I think that would be useful for people who want the easiest route to setting everything up. We're releasing a bunch of packages to PyPI now. Would it make sense to use those for released versions, so that we didn't need git versions in |
Which is where we have the problems anyway right? :)
We deploy off branches too, so we would need a workflow for that anyway. |
I think PyPi makes even more sense for a simple, stable deployment.
You all can figure out what you need for your development 😉 |
Yes, I think this is the best way. I thought it was possible now to |
@pnorman Filed related issues this weekend that we're working on. Thanks everyone for the feedback! |
Closing for inactivity. |
I have
tileserver v2.1.0
checked out and correspondinglytilequeue v1.8.0
andvector-datasource v1.4.0
checked out. I created a new python virtualenv, installed the requirements fromtileserver
, and then went totilequeue v1.8.0
andvector-datasource v1.4.0
and ranpython setup.py develop
(btw, should I not be passing adevelop
argument if using numbered releases?).When I run
python tileserver/__init__.py config.yaml
I getSolution
I noticed the very bottom of the tileserver
requirements.txt
file hasgit+https://github.com/mapzen/tilequeue@master#egg=tilequeue
. So I triedpip uninstall tilequeue
and gotand then
Did not produce that error.
The text was updated successfully, but these errors were encountered: