locationtech-labs / geopyspark

GeoTrellis for PySpark
Other
179 stars 59 forks source link

Fix geopyspark requirements #706

Closed lukaszKielar closed 5 years ago

lukaszKielar commented 5 years ago

This PR contains fix for issue #683.

Changes:

CC: @jbouffard I went through source code and it looks that rasterio isn't only package missing in requirements. I think we should add also GDAL, matplotlib, pyproj and colortools to list of requirements. ImportError's are raised inside some functions bodies which is not good idea especially when required libraries are not listed. What about taking all imports from functions to the beginning of modules?