<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Oct 2, 2013 at 10:12 AM, Benjamin Root <span dir="ltr"><<a href="mailto:ben.root@ou.edu" target="_blank">ben.root@ou.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote"><div><div class="h5">On Wed, Oct 2, 2013 at 11:43 AM, Charles R Harris <span dir="ltr"><<a href="mailto:charlesr.harris@gmail.com" target="_blank">charlesr.harris@gmail.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Stefan,<br><div class="gmail_extra"><br><br><div class="gmail_quote"><div>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><br>
On Tue, Oct 1, 2013 at 1:07 AM, Charles R Harris<br>
<<a href="mailto:charlesr.harris@gmail.com" target="_blank">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><font color="#888888"><br></font></span></blockquote><div><br></div></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></blockquote><div><br></div></div></div><div>So, just to be clear... what would happen if I had an array of floats between 0 and 1 inclusive and I cast that as a boolean using astype()?<br>


<br></div></div></div></div></blockquote><div><br></div><div>You get zeros and ones. I don't think that has changed. Of course, you can still mess things up using views ;)<br><br></div><div>Chuck <br></div></div><br>
</div></div>