geoplotlib icon indicating copy to clipboard operation
geoplotlib copied to clipboard

Adding requirements for pip to easily set up the development version of this repository

Open sudheesh001 opened this issue 9 years ago • 3 comments

While setting this up to play around with a few of the examples, I faced a few issues because of lack of dependencies that are required to run this file, one of which is pylab but which isn't mentioned in the installation document. Pylab error was thrown when running python graph-flights.py. It'd be great if this could be added to the documentation. Also a simpler way to set it up would be to use a requirements.txt file, so that in case developers use a virtualenv, they can get all the requirements to run geoplotlib with a one step installation by doing pip install -r requirements.txt , After the whole installation, the required requirements file on my dev environment. Most of these have been installed after pylab. It'd be great if this file could be present in the repo.

funcsigs==0.4
geoplotlib==0.3.0
matplotlib==1.4.3
mock==1.3.0
nose==1.3.7
numpy==1.9.3
pbr==1.8.0
pyglet==1.2.4
pyparsing==2.0.3
pyshp==1.2.3
python-dateutil==2.4.2
pytz==2015.6
scipy==0.16.0
six==1.9.0
wsgiref==0.1.2

sudheesh001 avatar Sep 30 '15 06:09 sudheesh001

Hi

pylab is installed as part of matplotlib.

The readme mentions that matplotlib is an optional requirement for the colormaps, but maybe it should be stated as mandatory.

The reason I do not want to use requirements.txt is that in my experience installing the scipy/numpy/matplotlib stack via pip leads to problems, so it's better if people install it separately.

andrea-cuttone avatar Sep 30 '15 15:09 andrea-cuttone

Hi @andrea-cuttone ,

That's great, I think its best to include matplotlib as a mandatory rather than an optional requirement, if that's the change, that'd be needed here, i'd be more than happy to send a PR to close this issue with the change from optional to mandatory as the patch.

The reason I do not want to use requirements.txt is that in my experience installing the scipy/numpy/matplotlib stack via pip leads to problems, so it's better if people install it separately.

I agree, I did face a few issues with the installation before but mostly due to clashing versions.

sudheesh001 avatar Oct 03 '15 20:10 sudheesh001

Thanks for your feedback. I would like to leave matplotlib as optional for now, as geoplotlib can indeed work without it if you manually specify the colors.

andrea-cuttone avatar Oct 04 '15 14:10 andrea-cuttone