Minimal version of swig?

automl / random_forest_run

Other

35 stars 24 forks source link

Minimal version of swig? #18

Closed mlindauer closed 7 years ago

mlindauer commented 7 years ago

Hi, on the nemo cluster, swig is installed in version 2.0.10 and the installation fails:

    swig -python -c++ -I${CMAKE_SOURCE_DIR}/include -I./include -o pyrfr/regression_wrap.cpp pyrfr/regression.i
    ./include/rfr/trees/binary_fanova_tree.hpp:329: Error: Syntax error in input(3).

On our machines, swig is running in version 3.0. So, I wanted ask whether there is a minimal required version of SWIG for pyrfr?

Cheers, Marius

mlindauer commented 7 years ago

btw: in case someone has a problem with swig and uses anaconda, here is an easy solution: conda install swig

sfalkner commented 7 years ago

Yes you will need Swig >= 3.0. They added a lot of C++11 support in that version. Thanks for the pointer to the swig-anaconda problem.

sfalkner commented 7 years ago

fixed via 170cfc86abc74a9bac834731270446b43571efcf