<div dir="ltr"><div dir="ltr" style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px">the 0s put into the array copy "arr" are not used in computation.  The _replace_nan call is used primarily to generate a mask of the NaNs and make sure it passes the mutation test.  I updated the unit tests to reflect negative values, which works.  (and the documentation should be cleaned up now)<div>

<br></div><div><a href="https://github.com/dfreese/numpy/compare/master...feature;nanmedian" target="_blank">https://github.com/dfreese/numpy/compare/master...feature;nanmedian</a><br></div><div><br></div></div></div><div class="gmail_extra">

<br><br><div class="gmail_quote">On Sun, Feb 16, 2014 at 9:52 AM, alex <span dir="ltr"><<a href="mailto:argriffi@ncsu.edu" target="_blank">argriffi@ncsu.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="">On Sun, Feb 16, 2014 at 12:13 PM, David Freese <<a href="mailto:dfreese@stanford.edu">dfreese@stanford.edu</a>> wrote:<br>
> Hi everyone,<br>
><br>
> I put together a np.nanmedian function to extend np.median to handle nans.<br>
> Could someone review this code and give me some feedback on it before I<br>
> submit a pull request for it?<br>
<br>
</div>It looks good to submit as a pull request but probably will need some<br>
changes like the mixed sign thing already mentioned, and I see mean<br>
vs. median copypaste remnants in the docstring.<br>
<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="http://mail.scipy.org/mailman/listinfo/numpy-discussion" target="_blank">http://mail.scipy.org/mailman/listinfo/numpy-discussion</a><br>
</div></div></blockquote></div><br></div>