Re: [Scikit-learn-general] setup script refering to .c

2013-03-05 Thread amueller
Exactly. Not only would you need cython, it also needs to be a recent version. people with older versions would get cryptic error messages, leading to frustrated users and busy mailing lists. Matthieu Brucher schrieb: >Hi, > >If I remember correctly, this is done to avoid an explicit Cython

Re: [Scikit-learn-general] setup script refering to .c

2013-03-05 Thread Matthieu Brucher
Hi, If I remember correctly, this is done to avoid an explicit Cython dependency. Cheers, Matthieu 2013/3/5 Kevin Kunzmann > Hi all, > > why are the .c files used as sources in the setup scripts and not the > respective .pyx files? It came to my mind when I tried to extend the > decision tre

[Scikit-learn-general] setup script refering to .c

2013-03-05 Thread Kevin Kunzmann
Hi all, why are the .c files used as sources in the setup scripts and not the respective .pyx files? It came to my mind when I tried to extend the decision tree splitting criteria and my changes where not compiled... Would it not be safer to generate the .c on the fly using cythonize()? Can an