[examples] pep8 fix E26* #3668
[examples] pep8 fix E26* #3668
Conversation
and minor modification of the output of autopep8 where appropriate
Signed-off-by: Thomas Hisch <[email protected]>
|
If it is too cumbersome to review I can split this PR into 2 parts |
| @@ -110,8 +110,6 @@ def __init__(self, sigma, fraction=0.5): | |||
| self.gauss_filter = GaussianFilter(sigma, alpha=1) | |||
| self.light_source = LightSource() | |||
| self.fraction = fraction | |||
| #hsv_min_val=0.5,hsv_max_val=0.9, | |||
| # hsv_min_sat=0.1,hsv_max_sat=0.1) | |||
There was a problem hiding this comment.
unused code even if the comments are removed
|
Are you doing any triaging on the examples while doing this? |
|
Yes, first I want to fix all the white-space issues (E2*) and then the rest. I order the white-space fix PRs such that each PR can be easily reviewed. The next planned big PR fixes E231 (missing whitespace after ‘,’ ) and the one (or two) after that all other white-space issues. |
|
Btw someone should probably fix the warnings in the examples when you are done with the pep8 fixes. It would be nice if building the examples did not raise all these warnings. I will hold off doing anything about this until you are done with the pep8 fixes to avoid merge conflicts. |
|
@jenshnielsen which warnings do you mean? |
|
Look at the travis output of building the docs. Several of the examples raises warnings when run |
|
Strange. I can reproduce those warnings. I'll try to fix them once we are done with the pep8 fixes. |
|
Any other comments? |
| @@ -22,7 +22,7 @@ | |||
| for i in range(len(DATA)): | |||
| (x,y) = DATA[i] | |||
| (dd, dl, r, dr, dp) = dash_style[i] | |||
| #print 'dashlen call', dl | |||
| #print('dashlen call %d' % dl) | |||
There was a problem hiding this comment.
Yay for fixing syntax in comments!
There was a problem hiding this comment.
Such stuff still has to be done manually :/
E261 at least two spaces before inline comment
E262 inline comment should start with ‘# ‘
E265 block comment should start with ‘# ‘
E266 too many leading ‘#’ for block comment
This PR also fixes some E211 (whitespace before ‘(‘ ) errors overseen by the autopep8/pep8 tool.