[Numpy-discussion] 1.8.0 branch reminder
Mon Aug 26 10:49:29 CDT 2013
On Mon, Aug 26, 2013 at 11:01 AM, Ralf Gommers <email@example.com>wrote:
> On Sun, Aug 18, 2013 at 6:36 PM, Charles R Harris <
> firstname.lastname@example.org> wrote:
>> On Sun, Aug 18, 2013 at 12:17 PM, Charles R Harris <
>> email@example.com> wrote:
>>> Just a reminder that 1.8.0 will be branched tonight. I've put up a big STY:
>>> PR <https://github.com/numpy/numpy/pull/3635> that removes trailing
>>> whitespace and fixes spacing after commas. I would like to apply before the
>>> branch, but it may cause merge difficulties down the line. I'd like
>>> feedback on that option.
>> I've also run autopep8 on the code and it does a nice job of cleaning up
>> things. It gets a little lost in deeply nested lists, but there aren't too
>> many of those. By default it doesn't fix spaces about operator (it seems).
>> I can apply that also if there is interest in doing so.
> Depends on how many lines of code it touches. For scipy we decided not to
> do this, because it would make "git blame" pretty much useless.
At some point, you just have to bite the bullet. Matplotlib has been doing
pep8 work for about a year now. We adopted very specific rules on how that
work was to be done (make pep8 only PRs, each pep8 PR would be for at most
one module at a time, etc). Yes, it does look like NelleV has taken over
the project, but the trade-off is readability. We even discovered a
non-trivial number of bugs this way. For a core library like NumPy that has
lots of very obscure-looking code that almost never gets changed, avoiding
PEP8 is problematic because it always becomes "Somebody else's problem".
Of course, it is entirely up to you, the devs, on what to do for NumPy and
SciPy, but that is what matplotlib is doing.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the NumPy-Discussion