the plotting module does not follow PEP8 (code formating)

[Krastanov]

The code there is a bit hard to read. It would be nice to have it refactored so it follows PEP8.

There are certain parts that are not pythonic but that's another problem needing more in-depth refactoring.

Original issue for #5914: http://code.google.com/p/sympy/issues/detail?id=2815
Original author: https://code.google.com/u/100157245271348669141/

[asmeurer]

I didn't notice anything particularly bad (at least there are spaces around the equals signs).  Can you be more specific about what the problems are?

Original comment: http://code.google.com/p/sympy/issues/detail?id=2815#c1
Original author: https://code.google.com/u/asmeurer@gmail.com/

[Krastanov]

Mainly 'if else' conditions that are all placed on the same line and are hard to read.

To find them one can use 

$ grep -E ': {1,}' *

It checks for ':' that are not the last character on the line for all files in the current directory.

**Labels:** CodeInDifficulty-Easy CodeInCategory-Code  

Original comment: http://code.google.com/p/sympy/issues/detail?id=2815#c2
Original author: https://code.google.com/u/100157245271348669141/

[asmeurer]

I see.  This could also be considered as a subset of issue 4555 .

**Status:** Accepted  

Referenced issues: #4555
Original comment: http://code.google.com/p/sympy/issues/detail?id=2815#c3
Original author: https://code.google.com/u/asmeurer@gmail.com/

[asmeurer]

**Labels:** EasyToFix  

Original comment: http://code.google.com/p/sympy/issues/detail?id=2815#c4
Original author: https://code.google.com/u/asmeurer@gmail.com/

[Krastanov]

**Labels:** CodeInImportedIntoSpreadsheet  

Original comment: http://code.google.com/p/sympy/issues/detail?id=2815#c5
Original author: https://code.google.com/u/100157245271348669141/

[asmeurer]

**Labels:** -codeinimportedintospreadsheet CodeInImportedIntoMelange  

Original comment: http://code.google.com/p/sympy/issues/detail?id=2815#c6
Original author: https://code.google.com/u/asmeurer@gmail.com/

[vperic]

This was fixed in: https://github.com/sympy/sympy/pull/735

**Status:** Fixed  

Original comment: http://code.google.com/p/sympy/issues/detail?id=2815#c7
Original author: https://code.google.com/u/108713607268198052411/