<div dir="ltr">I wouldn't have the keyword be "where", as that collides with the notion of "where" elsewhere in numpy.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jul 6, 2016 at 2:21 PM, Joseph Fox-Rabinovitz <span dir="ltr"><<a href="mailto:jfoxrabinovitz@gmail.com" target="_blank">jfoxrabinovitz@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I still think this function is useful. I have made a change so that it<br>
only accepts one array, as Marten suggested, making the API much<br>
cleaner than that of its siblings. The side on which the new<br>
dimensions will be added is configurable via the `where` parameter,<br>
which currently accepts 'before' and 'after', but can be changed to<br>
accept sequences or even dicts. The change also resulted in finding a<br>
bug in the masked array versions of the atleast functions, which the<br>
PR now fixes and adds regression tests for. If the devs do decide to<br>
discard this PR, I will of course submit the bug fix separately.<br>
<br>
    -Joe<br>
<div class="HOEnZb"><div class="h5"><br>
On Wed, Jul 6, 2016 at 1:43 PM, Stephan Hoyer <<a href="mailto:shoyer@gmail.com">shoyer@gmail.com</a>> wrote:<br>
> On Tue, Jul 5, 2016 at 10:06 PM, Nathaniel Smith <<a href="mailto:njs@pobox.com">njs@pobox.com</a>> wrote:<br>
>><br>
>> I don't know how typical I am in this. But it does make me wonder if the<br>
>> atleast_* functions act as an attractive nuisance, where new users take<br>
>> their presence as an implicit recommendation that they are actually a useful<br>
>> thing to reach for, even though they... aren't that. And maybe we should be<br>
>> recommending folk move away from them rather than trying to extend them<br>
>> further?<br>
><br>
> Agreed. I would avoid adding atleast_nd. We could discourage using<br>
> atleast_3d (certainly the behavior is indeed surprising), but I'm not sure<br>
> it's worth the trouble.<br>
><br>
</div></div><div class="HOEnZb"><div class="h5">> _______________________________________________<br>
> NumPy-Discussion mailing list<br>
> <a href="mailto:NumPy-Discussion@scipy.org">NumPy-Discussion@scipy.org</a><br>
> <a href="https://mail.scipy.org/mailman/listinfo/numpy-discussion" rel="noreferrer" target="_blank">https://mail.scipy.org/mailman/listinfo/numpy-discussion</a><br>
><br>
_______________________________________________<br>
NumPy-Discussion mailing list<br>
<a href="mailto:NumPy-Discussion@scipy.org">NumPy-Discussion@scipy.org</a><br>
<a href="https://mail.scipy.org/mailman/listinfo/numpy-discussion" rel="noreferrer" target="_blank">https://mail.scipy.org/mailman/listinfo/numpy-discussion</a><br>
</div></div></blockquote></div><br></div>