<div dir="ltr"><div>hi david,</div><div><br></div><div>I havnt run the code; but the _replace_nan(0) call worries me; especially considering that the unit tests seem to deal with positive numbers exclusively. Have you tested with mixed positive/negative inputs?</div>
<div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Feb 16, 2014 at 6:13 PM, David Freese <span dir="ltr"><<a href="mailto:dfreese@stanford.edu" target="_blank">dfreese@stanford.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi everyone,<div><br></div><div>I put together a np.nanmedian function to extend np.median to handle nans.  Could someone review this code and give me some feedback on it before I submit a pull request for it?</div>


<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>Thanks,</div>
<div>David</div>

</div>
<br>_______________________________________________<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>
<br></blockquote></div><br></div>