[SciPy-Dev] ndimage and from __future__ import division

Stéfan van der Walt stefan@sun.ac...
Tue Apr 27 09:30:12 CDT 2010


Hi Chris

On 23 April 2010 03:31, Christopher Hanley <chanley@stsci.edu> wrote:
> In preparation for our next release we are preparing to support Python
> 2.6 with __future__ division.  Since I was one of the last STScI folks
> to touch scipy I was tasked with looking into how ndimage would work
> with "from __future__ import division" added.   All of the existing
> tests pass with the exception of 2:

I looked through all the division operators in ndimage, and it seems
Peter consistently used "//" for integer division.  Since the C code
is not influenced, you should be safe.

The only case I wasn't sure about is this line from morphology.py:

    pos = [ni * (structure.shape[ii] / 2) for ii in range(len(shape))]

Since you now get two errors, I suspect the problem might be with the
way the unit tests are constructed (I haven't checked how they used
division).

Regards
Stéfan


More information about the SciPy-Dev mailing list