<br><br><div class="gmail_quote">On Thu, Jun 14, 2012 at 10:17 AM, Nathaniel Smith <span dir="ltr"><<a href="mailto:njs@pobox.com" target="_blank">njs@pobox.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Wed, Jun 6, 2012 at 11:08 PM, Nathaniel Smith <<a href="mailto:njs@pobox.com">njs@pobox.com</a>> wrote:<br>
> Just submitted this pull request for discussion:<br>
>  <a href="https://github.com/numpy/numpy/pull/297" target="_blank">https://github.com/numpy/numpy/pull/297</a><br>
><br>
> As per earlier discussion on the list, this PR attempts to remove<br>
> exactly and only the maskna-related code from numpy mainline:<br>
>  <a href="http://mail.scipy.org/pipermail/numpy-discussion/2012-May/062417.html" target="_blank">http://mail.scipy.org/pipermail/numpy-discussion/2012-May/062417.html</a><br>
><br>
> The suggestion is that we merge this to master for the 1.7 release,<br>
> and immediately "git revert" it on a branch so that it can be modified<br>
> further without blocking the release.<br>
><br>
> The first patch does the actual maskna removal; the second and third<br>
> rearrange things so that PyArray_ReduceWrapper does not end up in the<br>
> public API, for reasons described therein.<br>
><br>
> All tests pass with Python 2.4, 2.5, 2.6, 2.7, 3.1, 3.2 on 64-bit<br>
> Ubuntu. The docs also appear to build. Before I re-based this I also<br>
> tested against Scipy, matplotlib, and pandas, and all were fine.<br>
<br>
</div>While it's tempting to think that the lack of response to this<br>
email/PR indicates that everyone now agrees with me about how to<br>
proceed with the NA work, I'm for some reason unconvinced...<br>
<br>
Any objections to merging this?<br>
<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br>Travis needs to sign off on this one.<br><br>Chuck <br></div><br></div>