search

cesium-ml / cesium

Machine Learning Time-Series Platform
Other
674 stars 101 forks source link

Error: Microsoft Visual C++ 14.0 is required. #290

Closed KevinFumbles closed 4 years ago

KevinFumbles commented 4 years ago

I'm not sure if I don't completely understand the build process, but when trying to build the latest version (v0.9.12) from source on Windows, pip errors with an error message stating that Microsoft Visual C++ 14.0 is required. I believe this is not correct since it is explicitly stated in the README that MSVC does not have the compiler required and that an alternative (e.g. MinGW) is necessary to build from source. I have seen this behavior on a fresh installation of Anaconda 5.2.0 as well as 2020.02.

Can this error be changed from suggesting to install MSVC to MinGW(-w64)?

I didn't think providing logs was necessary in this case, but if requested I can provide it.

acrellin commented 4 years ago

@KevinFumbles are you attempting to build from source (https://github.com/cesium-ml/cesium#installation-from-source) or just doing pip install cesium? Please follow the instructions to build from source here, after ensuring you have MinGW installed: https://github.com/cesium-ml/cesium#installation-from-source Let us know if that works for you.

KevinFumbles commented 4 years ago

Sorry, I misspoke earlier. For this scenario, it was just a pip install cesium that lead to the error about needing to install MSVC.

To clarify the source build instructions, do you need Cython AND MinGW, or does one suffice? If only one, am I correct in assuming that Cython would be the "correct" way?

acrellin commented 4 years ago

Cython runs on top of MinGW, so you need to install both.

acrellin commented 4 years ago

@KevinFumbles https://cython.readthedocs.io/en/latest/src/tutorial/appendix.html#appendix-installing-mingw-on-windows

acrellin commented 4 years ago

We've updated the installation instructions to be more clear here: https://github.com/cesium-ml/cesium/pull/291

KevinFumbles commented 4 years ago

Ok that makes sense. With that said, I have followed the build instructions, and I was able to install v0.9.6 but v0.9.12 fails with the following output. Since my original issue isn't relevant if the correct build procedure is taken, I'll go ahead and close this. failed.log

acrellin commented 4 years ago

Were you able to successfully install from source following the new instructions?

On Wed, Apr 1, 2020, 12:26 PM Kevin notifications@github.com wrote:

Closed #290 https://github.com/cesium-ml/cesium/issues/290.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/cesium-ml/cesium/issues/290#event-3189355305, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABXFGTJ67SHIJWP27P7RCD3RKOIMRANCNFSM4LZFGF7Q .