<br><br><div class="gmail_quote">On Fri, May 6, 2011 at 12:57 PM, Derek Homeier <span dir="ltr"><<a href="mailto:derek@astro.physik.uni-goettingen.de">derek@astro.physik.uni-goettingen.de</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"><br>
On 6 May 2011, at 07:53, Ralf Gommers wrote:<br>
<br>
><br>
> >> Looks okay, and I agree that it's better to fix it now. The timing<br>
> >> is a bit unfortunate though, just after RC2. I'll have closer look<br>
> >> tomorrow and if it can go in, probably tag RC3.<br>
> >><br>
> >> If in the meantime a few more people could test this, that would be<br>
> >> helpful.<br>
> >><br>
> >> Ralf<br>
> ><br>
> > I agree, wish I had time to push this before rc2. I could add the<br>
> > explanatory comments<br>
> > mentioned above and switch to use the atleast_[12]d() solution, test<br>
> > that and push it<br>
> > in a couple of minutes, or should I better leave it as is now for<br>
> > testing?<br>
><br>
> Quick follow-up: I just applied the above changes, added some tests to<br>
> cover Ben's test cases and tested this with 1.6.0rc2 on OS X 10.5<br>
> i386+ppc<br>
> + 10.6 x86_64 (Python2.7+3.2). So I'd be ready to push it to my repo<br>
> and do<br>
> my (first) pull request...<br>
><br>
> Go ahead, I'll have a look at it tonight. Thanks for testing on<br>
> several Pythons, that definitely helps.<br>
<br>
<br>
</div>Done, the request only appears on my repo<br>
<div class="im"><a href="https://github.com/dhomeier/numpy/" target="_blank">https://github.com/dhomeier/numpy/</a><br>
<br>
</div>is that correct? If someone could test it on Linux and Windows as<br>
well...<br></blockquote><div><br>Committed, thanks for all the work.<br><br>The pull request was in the wrong place, that's a minor flaw in the github UI. After you press "Pull Request" you need to read the small print to see where it's going. <br>
<br>Cheers,<br>Ralf<br> <br></div></div>