<br><br><div class="gmail_quote">On Mon, Apr 4, 2011 at 9:33 AM, Charles R Harris <span dir="ltr"><<a href="mailto:charlesr.harris@gmail.com">charlesr.harris@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<br><br><div class="gmail_quote"><div><div></div><div class="h5">On Mon, Apr 4, 2011 at 8:57 AM, Paul Anton Letnes <span dir="ltr"><<a href="mailto:paul.anton.letnes@gmail.com" target="_blank">paul.anton.letnes@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div><div></div><div><br>
On 4. apr. 2011, at 16.42, Charles R Harris wrote:<br>
<br>
><br>
><br>
> On Mon, Apr 4, 2011 at 8:29 AM, Paul Anton Letnes <<a href="mailto:paul.anton.letnes@gmail.com" target="_blank">paul.anton.letnes@gmail.com</a>> wrote:<br>
><br>
> On 4. apr. 2011, at 15.34, Charles R Harris wrote:<br>
><br>
> ><br>
> ><br>
> > On Sun, Apr 3, 2011 at 4:35 AM, Paul Anton Letnes <<a href="mailto:paul.anton.letnes@gmail.com" target="_blank">paul.anton.letnes@gmail.com</a>> wrote:<br>
> > Hi.<br>
> ><br>
> > When looking at the loadtxt/savetxt tickets, I noticed that the 're' module is imported in an odd place. I therefore suggest that this import is moved to the top of the file, in order to gather these as much as possible. I find the code easier to read then. After all, there is no 'try / catch' or similar to check if the module exists. See patch below. I do not believe any tests or tickets are needed - correct me if I am wrong.<br>


> ><br>
> > Cheers,<br>
> > Paul.<br>
> ><br>
> ><br>
> > --- a/numpy/lib/npyio.py        Sat Apr 02 20:19:55 2011 -0600<br>
> > +++ b/numpy/lib/npyio.py        Sun Apr 03 12:30:02 2011 +0200<br>
> > @@ -6,6 +6,7 @@<br>
> >  import format<br>
> >  import sys<br>
> >  import os<br>
> > +import re<br>
> >  import sys<br>
> >  import itertools<br>
> >  import warnings<br>
> > @@ -956,7 +957,6 @@<br>
> >         if own_fh:<br>
> >             fh.close()<br>
> ><br>
> > -import re<br>
> >  def fromregex(file, regexp, dtype):<br>
> >     """<br>
> >     Construct an array from a text file, using regular expression parsing.<br>
> ><br>
> > If you want to see a lot of other small things for cleanups, run pyflakes or pylint on the files in numpy/lib<br>
> ><br>
> > Chuck<br>
> > _______________________________________________<br>
> > NumPy-Discussion mailing list<br>
> > <a href="mailto:NumPy-Discussion@scipy.org" target="_blank">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>
> Are you suggesting that I should do this and submit one or more patches?<br>
><br>
> That would be great, and a good way to get into numpy development.<br>
><br>
><br>
> By the way - what is a suggested form of submitting patches? Using e-mail seems a bit clumsy to me.<br>
><br>
><br>
> The best thing would be to setup to do your work on github and then issue pull requests, see the directions here: <a href="http://docs.scipy.org/doc/numpy/dev/gitwash/development_setup.html" target="_blank">http://docs.scipy.org/doc/numpy/dev/gitwash/development_setup.html</a>. If you are on linux or mac getting set up should be pretty easy. Windows may be trickier, I don't have experience doing development on that platform.<br>


<br>
</div></div>I'm on a mac, and I've got git. The skills to use it will have to grow with time.<br>
<br>
I submitted a pull request on this particular patch - let me know if I got it right!<br>
<br></blockquote></div></div><div><br>Looked good, I went ahead and applied it. Mind, we aren't always so responsive ;)<br><br></div></div></blockquote><div><br>One other thing, you should do even the trivial stuff in a branch, keeping the master branch of your numpy fork updated from the numpy repository directly. That way your diffs will based on numpy instead of your own fork.<br>
<br>Chuck <br></div><br></div>