<br><br><div class="gmail_quote">On Thu, Jun 14, 2012 at 5:17 PM, Nathaniel Smith <span dir="ltr">&lt;<a href="mailto:njs@pobox.com" target="_blank">njs@pobox.com</a>&gt;</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 &lt;<a href="mailto:njs@pobox.com">njs@pobox.com</a>&gt; wrote:<br>
&gt; Just submitted this pull request for discussion:<br>
&gt;  <a href="https://github.com/numpy/numpy/pull/297" target="_blank">https://github.com/numpy/numpy/pull/297</a><br>
&gt;<br>
&gt; As per earlier discussion on the list, this PR attempts to remove<br>
&gt; exactly and only the maskna-related code from numpy mainline:<br>
&gt;  <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>
&gt;<br>
&gt; The suggestion is that we merge this to master for the 1.7 release,<br>
&gt; and immediately &quot;git revert&quot; it on a branch so that it can be modified<br>
&gt; further without blocking the release.<br>
&gt;<br>
&gt; The first patch does the actual maskna removal; the second and third<br>
&gt; rearrange things so that PyArray_ReduceWrapper does not end up in the<br>
&gt; public API, for reasons described therein.<br>
&gt;<br>
&gt; All tests pass with Python 2.4, 2.5, 2.6, 2.7, 3.1, 3.2 on 64-bit<br>
&gt; Ubuntu. The docs also appear to build. Before I re-based this I also<br>
&gt; tested against Scipy, matplotlib, and pandas, and all were fine.<br>
<br>
</div>While it&#39;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&#39;m for some reason unconvinced...<br>
<br>
Any objections to merging this?<br></blockquote><div><br>No objection, but could you wait for this WE ? I am in the middle of setting up a buildbot for windows for numpy (for both mingw and MSVC compilers), and that would be a good way to test it.<br>
<br>David<br></div></div>