<div class="gmail_quote">On Thu, Mar 31, 2011 at 8:12 PM, Eli Stevens (Gmail) <span dir="ltr"><<a href="mailto:wickedgrey@gmail.com">wickedgrey@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
On Thu, Mar 31, 2011 at 11:45 AM, Ralf Gommers<br>
<div class="im"><<a href="mailto:ralf.gommers@googlemail.com">ralf.gommers@googlemail.com</a>> wrote:<br>
</div><div class="im">> On Thu, Mar 31, 2011 at 8:37 PM, Eli Stevens (Gmail)<br>
> <<a href="mailto:wickedgrey@gmail.com">wickedgrey@gmail.com</a>> wrote:<br>
</div><div class="im">>> Next up for me is to get a patch onto the CPython issue tracker, but<br>
>> as soon as I have that done, I'll start working on adding unit tests<br>
>> to my numpy fork.  I anticipate being able to get that done today.<br>
><br>
> Sounds good.<br>
<br>
</div>CPython issue here:<br>
<a href="http://bugs.python.org/issue11734" target="_blank">http://bugs.python.org/issue11734</a><br>
<br>
My numpy changes so far:<br>
<a href="https://github.com/wickedgrey/numpy/compare/23965b6b665985390d7a...maintenance%2F1.6.x" target="_blank">https://github.com/wickedgrey/numpy/compare/23965b6b665985390d7a...maintenance%2F1.6.x</a><br>
<br>
>From my last commit message:<br>
"Note that my changes to _internal were mostly of the "make it work"<br>
variety; I'm not really understanding the wider scope.  Very much<br>
needs review by wiser eyes."<br>
<br>
I'd also appreciate some feedback on the unit test coverage; the<br>
test_roundtrip_half function uncovered a problem that the other two<br>
didn't (which surprised me), and I'd like to make sure that there<br>
aren't more gaps in the tests that I've written.<br></blockquote><div><br></div><div>Your changes look fine to me, and I think the tests are reasonable as is. I would suggest tidying up your commit messages a bit, creating an enhancement bug in the bug tracker, and making a pull request. Some info about the pull requests is here:</div>
<div><br></div><div><meta http-equiv="content-type" content="text/html; charset=utf-8"><a href="http://docs.scipy.org/doc/numpy/dev/gitwash/development_workflow.html">http://docs.scipy.org/doc/numpy/dev/gitwash/development_workflow.html</a></div>
<div><br></div><div>-Mark</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
Thanks for all the help!<br>
<font color="#888888">Eli<br>
</font><div><div></div><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>