<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Tue, Nov 6, 2018 at 8:09 PM Stephan Hoyer <<a href="mailto:shoyer@gmail.com">shoyer@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Tue, Nov 6, 2018 at 6:08 PM Stefan van der Walt <<a href="mailto:stefanv@berkeley.edu" target="_blank">stefanv@berkeley.edu</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sun, 04 Nov 2018 17:16:12 -0800, Stephan Hoyer wrote:<br>
> On Sun, Nov 4, 2018 at 10:32 AM Marten van Kerkwijk <<br>
> <a href="mailto:m.h.vankerkwijk@gmail.com" target="_blank">m.h.vankerkwijk@gmail.com</a>> wrote:<br>
> <br>
> > For `__array_function__`, there was some discussion in<br>
> > <a href="https://github.com/numpy/numpy/issues/12225" rel="noreferrer" target="_blank">https://github.com/numpy/numpy/issues/12225</a> that for 1.16 we might want<br>
> > to follow after all Nathaniel's suggestion of using an environment variable<br>
> > or so to opt in (since introspection breaks on python2 with our wrapped<br>
> > implementations).  Given also the possibly significant hit in performance,<br>
> > this may be the best option.<br>
> > All the best,<br>
> <br>
> I am also leaning towards this right now, depending on how long we plan to<br>
> wait for releasing 1.16. It will take us at least a little while to sort<br>
> out performance issues for __array_function__, I'd guess at least a few<br>
> weeks. Then a blocker still might turn up during the release candidate<br>
> process (though I think we've found most of the major bugs / downstream<br>
> issues already through tests on NumPy's dev branch).<br>
<br>
Just to make sure I understand correctly: the suggestion is to use an<br>
environment variable to temporarily toggle the feature, but the plan in<br>
the long run will be to have it enabled all the time, correct?<br></blockquote><div><br></div></div></div><div dir="ltr"><div class="gmail_quote"><div>Yes, exactly. __array_function__ would be opt-in only for 1.16 but enabled by default for 1.17. </div></div></div></blockquote><div><br></div><div>I've gone ahead and made a pull request making __array_function__ opt-in only for now:</div><div><a href="https://github.com/numpy/numpy/pull/12362">https://github.com/numpy/numpy/pull/12362</a> </div></div></div>