Re: [Scikit-learn-general] I messed up or master severely broken

2012-10-28 Thread Gael Varoquaux
On Sun, Oct 28, 2012 at 10:03:05PM +, Andreas Mueller wrote: > HMM still broken: >File "_hmmc.pyx", line 24, in sklearn._hmmc._forward > (sklearn/_hmmc.c:1339) > ValueError: Buffer dtype mismatch, expected 'dtype_t' but got 'float' I tried fixing: https://github.com/scikit-learn/scikit-le

Re: [Scikit-learn-general] I messed up or master severely broken

2012-10-28 Thread Andreas Mueller
On 10/28/2012 09:56 PM, Gael Varoquaux wrote: > On Sun, Oct 28, 2012 at 09:38:31PM +, Andreas Mueller wrote: >> Thanks for the quick fix :) > Can you now build master? > > It works for me (and it's time for me to go to bed :} ). > > Gaël > > PS: looks like we need better/more buildbots > HMM st

Re: [Scikit-learn-general] I messed up or master severely broken

2012-10-28 Thread Gael Varoquaux
On Sun, Oct 28, 2012 at 09:38:31PM +, Andreas Mueller wrote: > Thanks for the quick fix :) Can you now build master? It works for me (and it's time for me to go to bed :} ). Gaël PS: looks like we need better/more buildbots --

Re: [Scikit-learn-general] I messed up or master severely broken

2012-10-28 Thread Andreas Mueller
On 10/28/2012 09:34 PM, Lars Buitinck wrote: > 2012/10/28 Andreas Mueller : >> Sorry, thought unitc was the C-inferface. >> It is 4294967295, which is 2 ** 32 - 1, and doesn't fit in an int. >> If the random seed is a signed int (which it seems to be), it is weird >> that it works on 64bit, isn't i

Re: [Scikit-learn-general] I messed up or master severely broken

2012-10-28 Thread Lars Buitinck
2012/10/28 Andreas Mueller : > Sorry, thought unitc was the C-inferface. > It is 4294967295, which is 2 ** 32 - 1, and doesn't fit in an int. > If the random seed is a signed int (which it seems to be), it is weird > that it works on 64bit, isn't it? I already pushed a fix. uintc is indeed the C t

Re: [Scikit-learn-general] I messed up or master severely broken

2012-10-28 Thread Andreas Mueller
On 10/28/2012 09:15 PM, Andreas Mueller wrote: > On 10/28/2012 09:10 PM, Lars Buitinck wrote: >> 2012/10/28 Andreas Mueller : >>> In the SVM I get >>> File "mtrand.pyx", line 868, in mtrand.RandomState.randint >>> (numpy/random/mtrand/mtrand.c:5921) >>> OverflowError: Python int too large to c

Re: [Scikit-learn-general] I messed up or master severely broken

2012-10-28 Thread Andreas Mueller
On 10/28/2012 09:10 PM, Lars Buitinck wrote: > 2012/10/28 Andreas Mueller : >> In the SVM I get >> File "mtrand.pyx", line 868, in mtrand.RandomState.randint >> (numpy/random/mtrand/mtrand.c:5921) >> OverflowError: Python int too large to convert to C long >> >> Can others reproduce? >> I guess

Re: [Scikit-learn-general] I messed up or master severely broken

2012-10-28 Thread Lars Buitinck
2012/10/28 Andreas Mueller : > In the SVM I get >File "mtrand.pyx", line 868, in mtrand.RandomState.randint > (numpy/random/mtrand/mtrand.c:5921) > OverflowError: Python int too large to convert to C long > > Can others reproduce? > I guess the SVM is a 32bit/64bit issue. It took me quite a wh

Re: [Scikit-learn-general] I messed up or master severely broken

2012-10-28 Thread Alexandre Gramfort
hi Andy, it's also broken on my python 32 for the same reasons… you're not alone ! Alex On Sun, Oct 28, 2012 at 9:44 PM, Kenneth C. Arnold wrote: > Btw, if at some point you do need to look at diffs for the cython generated > code, temporarily removing the gitattributes file should suffice. Yo

Re: [Scikit-learn-general] I messed up or master severely broken

2012-10-28 Thread Andreas Mueller
On 10/28/2012 08:44 PM, Kenneth C. Arnold wrote: > > Btw, if at some point you do need to look at diffs for the cython > generated code, temporarily removing the gitattributes file should > suffice. You probably don't even have to commit, tho be aware of doing > it by accident :) > > Thanks, goo

Re: [Scikit-learn-general] I messed up or master severely broken

2012-10-28 Thread Kenneth C. Arnold
Btw, if at some point you do need to look at diffs for the cython generated code, temporarily removing the gitattributes file should suffice. You probably don't even have to commit, tho be aware of doing it by accident :) -Ken On Oct 28, 2012 4:32 PM, "Andreas Mueller" wrote: > Hey everybody. >

[Scikit-learn-general] I messed up or master severely broken

2012-10-28 Thread Andreas Mueller
Hey everybody. I had several problems when recompiling master today. Not sure if I messed up some of my packages or what. I get errors in the Cython parts of the SVM (if I undo Lars' work it works again) and the HMM - I just realized that the other thing I wanted to complain about was me breaking

Re: [Scikit-learn-general] random forest question

2012-10-28 Thread Andreas Mueller
On 10/28/2012 01:52 AM, Joseph Turian wrote: > Actually, it turns out I was incorrect. > > According to the docs: > http://scikit-learn.org/stable/modules/ensemble.html#forests-of-randomized-trees > > "each tree in the ensemble is built from a sample drawn with > replacement (i.e., a bootstrap samp