RJT1990 / pyflux

Open source time series library for Python
BSD 3-Clause "New" or "Revised" License
2.11k stars 240 forks source link

Compilation error #31

Closed masdeseiscaracteres closed 7 years ago

masdeseiscaracteres commented 8 years ago

Hi, First of all, I would like to congratulate/thank you for the amazing job and the great effort devoted to compile the tools in pyflux. However, I have not had the chance to test it out since I am getting a compile-time error when installing it on Windows:

 gas_recursions.obj : warning LNK4197: export 'PyInit_gas_recursions' specified multiple times; using first specification
     Creating library build\temp.win-amd64-3.5\Release\pyflux\gas\gas_recursions.cp35-win_amd64.lib and object build\temp.win-amd64-3.5\Release\pyflux\gas\gas_recursions.cp35-win_amd64.exp
  Generating code
  c:\users\user\appdata\local\temp\pip-build-m71zj04b\pyflux\pyflux\gas\gas_recursions.c(2167) : fatal error C1001: An internal error has occurred in the compiler.
  (compiler file 'f:\dd\vctools\compiler\utc\src\p2\main.c', line 255)

Any clue? I will try to have a look at it when I have some spare time but I thought this error might be familiar to someone else out there. For those interested, the complete output is here.

RJT1990 commented 8 years ago

Thanks for reporting; will have a look later today

RJT1990 commented 8 years ago

Hi masdeseiscaracteres, could you try reinstalling numpy and see if that works for you?

masdeseiscaracteres commented 8 years ago

From a command line with admin rights I reinstalled numpy as you suggested:

pip install --upgrade --force-reinstall numpy

but I am still getting the same error.

Then, I tried downloading the source code, cythonizing (previously I added "____check_build" and "tools" directories from cython_template) and building, which worked like a charm. More specifically, I did:

python setup.py build_ext -i
python setup.py install

Still wondering what is wrong with the PyPI distribution.

RJT1990 commented 7 years ago

Closing this issue because I'm not sure it's still relevant for the current release, but please create an issue if this problem is persisting.