<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Sep 7, 2013 at 7:56 PM, Charles R Harris <span dir="ltr"><<a href="mailto:charlesr.harris@gmail.com" target="_blank">charlesr.harris@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Hi All,<br><br></div>I've been doing some PEP8 work using autopep8. One problem that has turned up is that the default behavior of autopep8 is version dependent. I'd like to put a script in numpy tools that runs autopep8 with some features disabled, namely<br>


<ol><li>E226 -- puts spaces around arithmetic operators (+, -, *, /, **).<br></li><li>E241 -- allows only single spaces after ','</li></ol></div></blockquote><div>Something we have done in matplotlib is that we have made PEP8 a part of
 the tests. We are transitioning, but the idea is that eventually, with 
Travis, all pull requests will get PEP8 checked. I am very leary of 
automatic PEP8-ing. I would rather have the tests fail and let me 
manually fix it rather than have code automatically changed.<br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><p>The first leaves expression formatting in the hands of the coder and avoids things like "2 ** 3". The second allows array entries to be vertically aligned, which can be useful in clarifying the values used in tests. A few other things that might need decisions:</p>


<ol><li><span style="font-family:courier new,monospace">[:,:, 2]</span> or<span style="font-family:courier new,monospace"> [:, :, 2]</span></li><li><span style="font-family:courier new,monospace"><font face="arial,helvetica,sans-serif">Blank line before first function after <span style="font-family:courier new,monospace">class</span></font> Foo():<br>

</span></li></ol></div></blockquote><div>For the first one, I prefer spaces. For the second one, I prefer no blank lines.<br><br></div><div>Cheers!<br>Ben Root<br></div></div></div></div>