On Mon, Sep 24, 2012 at 5:47 PM, Charles R Harris <charlesr.harris@gmail.com> wrote:
On Mon, Sep 24, 2012 at 2:25 PM, Frédéric Bastien <nouiz@nouiz.org> wrote:
Hi,
I tested this new beta on Theano and discovered an interface change that was not there in the beta 1.
New behavior: numpy.ndindex().next() (0,)
Old behavior: numpy.ndindex().next() ()
This break some Theano code that look like this:
import numpy shape=() out_shape=[12] random_state=numpy.random.RandomState()
out = numpy.zeros(out_shape, int) for i in numpy.ndindex(*shape): out[i] = random_state.permutation(5)
I suppose this is an regression as the only mention of ndindex in the first email of this change is that it is faster.
I think this problem has been brought up on the list. It is interesting that it turned up after the first beta. Could you do a bisection to discover which commit is responsible?
I'll check that. Do I need to reinstall numpy from scratch everytimes or is there a better way to do that? Fred