You mean complying to PEP8 in this line would be ugly? I agree with you. I
think we're usually reasonable about these things. In other words, no need
to put the ugly spaces in here. Do you have any sense for why the test is
currently failing on Travis?

https://travis-ci.org/nipy/dipy/jobs/26140067


On Tue, May 27, 2014 at 8:25 AM, Samuel St-Jean <notifications@github.com>wrote:

> In dipy/denoise/noise_estimate.py:
>
> > +                   16: 0.5900487123737876,
> > +                   32: 0.5641772300866416,
> > +                   64: 0.5455611840489607,
> > +                  128: 0.5322811923303339}
> > +
> > +    if N in opt_quantile:
> > +        q = opt_quantile[N]
> > +    else:
> > +        q = 0.5
> > +
> > +    # Initial estimation of sigma
> > +    denom = np.sqrt(2 * _inv_nchi_cdf(N, 1, q))
> > +    m = np.percentile(data, q*100) / denom
> > +    phi = np.arange(1, l + 1) * m/l
> > +    K = data.shape[-1]
> > +    sum_m2 = np.sum(data**2, axis=-1)
>
> Having the exponent float with a space seems ugly to me.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/nipy/dipy/pull/337/files#r13083485>
> .
>