<div dir="ltr">Hi Stefan,<br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Oct 2, 2013 at 9:29 AM, Stéfan van der Walt <span dir="ltr"><<a href="mailto:stefan@sun.ac.za" target="_blank">stefan@sun.ac.za</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Chuck<br>
<div class="im"><br>
On Tue, Oct 1, 2013 at 1:07 AM, Charles R Harris<br>
<<a href="mailto:charlesr.harris@gmail.com">charlesr.harris@gmail.com</a>> wrote:<br>
>> I'll bet the skimage problems come from<br>
>> <a href="https://github.com/numpy/numpy/pull/3811" target="_blank">https://github.com/numpy/numpy/pull/3811</a>. They may be doing something<br>
>> naughty...<br>
>><br>
><br>
> Reverting that commit fixes those skimage failures. However, there are a<br>
> number of python2.7 failures that look pretty strange.<br>
<br>
</div>What is the exact change in behavior with that PR?  I'm trying to<br>
figure out what skimage does wrong in this case.<br>
<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>The current master, and reverted for the 1.8 release only, is stricter about np.bool only taking values 0 or 1. Apparently the convolve returns boolean (I haven't checked) for boolean input, and consequently the check if the return value matches the number of 1 elements in the convolution kernel will fail when that number is greater than one. That is why the proposed fix is to view the boolean as uint8 instead. Note that out=(boolean) will still cause problems.<br>
<br></div><div>Chuck <br></div><br></div></div></div>