search

Esri / raster-deep-learning

ArcGIS built-in python raster functions for deep learning to get you started fast.
Apache License 2.0
190 stars 88 forks source link

Install from env_setup.bat missing cython #13

Closed tim-mcg closed 5 years ago

tim-mcg commented 5 years ago

When installing the environment using the provided env_setup.bat file an error is shown :

object-detection 0.1 requires Cython>=0.28.1, which is not installed. mkl-random 1.0.1 requires cython, which is not installed. mkl-fft 1.0.4 requires cython, which is not installed. cftime 1.0.0b1 requires cython, which is not installed.

This is easily fixed by modifying the following line in the batch file from:

CALL "%SCRIPTS_DIR%conda.exe" install -c anaconda %TF_PACKAGE% keras scikit-image --yes

To:

CALL "%SCRIPTS_DIR%conda.exe" install -c anaconda cython %TF_PACKAGE% keras scikit-image --yes

lingtangraster commented 5 years ago

@tim-mcg Thanks for the good suggestion. This requirement is newly introduced after conda install upgrading to TensorFlow 1.13. We can certainly make the suggested change. However, as you may also realize, it is unrealistic to address all Python libraries dependency issues for users, because it is also related to the users' specific Python environments. The users are responsible to setup a workable Python Env. towards the third party deep learning framework APIs, such as TensorFlow. The environment setup batch file (env_setup.bat) provided in this repo is for general use purpose, which provides a good start for the users.