Skip to content

Conversation

samuelgarcia
Copy link
Contributor

@samuelgarcia samuelgarcia commented Nov 16, 2018

Fix warnings see #602

  • fromstring warnings
  • DeprecationWarning: invalid escape sequence

@pep8speaks
Copy link

pep8speaks commented Nov 16, 2018

Hello @samuelgarcia! Thanks for updating the PR.

Line 281:68: E712 comparison to False should be 'if cond is False:' or 'if not cond:'
Line 1111:13: W504 line break after binary operator
Line 1270:17: W504 line break after binary operator
Line 1280:17: W504 line break after binary operator
Line 1281:17: W504 line break after binary operator
Line 1313:17: W504 line break after binary operator
Line 1316:17: W504 line break after binary operator
Line 1339:17: W504 line break after binary operator
Line 1413:21: W504 line break after binary operator
Line 1454:21: W504 line break after binary operator
Line 1661:17: E129 visually indented line with same indent as next logical line
Line 1661:17: W503 line break before binary operator
Line 1724:38: E126 continuation line over-indented for hanging indent
Line 1829:21: W504 line break after binary operator
Line 1830:21: W504 line break after binary operator
Line 2011:46: E126 continuation line over-indented for hanging indent
Line 2013:42: E126 continuation line over-indented for hanging indent
Line 2015:42: E126 continuation line over-indented for hanging indent
Line 2030:44: W504 line break after binary operator
Line 2030:48: E126 continuation line over-indented for hanging indent
Line 2031:44: E131 continuation line unaligned for hanging indent
Line 2037:44: W504 line break after binary operator
Line 2037:48: E126 continuation line over-indented for hanging indent
Line 2038:44: E131 continuation line unaligned for hanging indent
Line 2054:44: E126 continuation line over-indented for hanging indent
Line 2056:44: E126 continuation line over-indented for hanging indent
Line 2562:100: E501 line too long (104 > 99 characters)

Line 533:10: W605 invalid escape sequence '\d'
Line 533:16: W605 invalid escape sequence '\d'
Line 533:20: W605 invalid escape sequence '\D'
Line 533:23: W605 invalid escape sequence '\D'
Line 597:13: E722 do not use bare 'except'
Line 1191:25: W504 line break after binary operator
Line 1673:9: E722 do not use bare 'except'
Line 1681:9: E722 do not use bare 'except'
Line 1788:9: E722 do not use bare 'except'
Line 3531:21: W503 line break before binary operator
Line 3965:36: W504 line break after binary operator

Line 33:33: W504 line break after binary operator
Line 34:33: W504 line break after binary operator
Line 152:79: W291 trailing whitespace
Line 288:17: W504 line break after binary operator
Line 289:17: E129 visually indented line with same indent as next logical line
Line 377:17: W504 line break after binary operator
Line 378:17: E129 visually indented line with same indent as next logical line

Line 35:1: E402 module level import not at top of file
Line 37:1: E402 module level import not at top of file
Line 39:1: E402 module level import not at top of file
Line 40:1: E402 module level import not at top of file
Line 42:1: E402 module level import not at top of file
Line 43:1: E402 module level import not at top of file
Line 45:1: E402 module level import not at top of file
Line 46:1: E402 module level import not at top of file
Line 270:21: W503 line break before binary operator
Line 272:21: W503 line break before binary operator
Line 274:21: W503 line break before binary operator
Line 276:25: W503 line break before binary operator
Line 532:21: E126 continuation line over-indented for hanging indent
Line 532:21: W504 line break after binary operator
Line 540:21: E126 continuation line over-indented for hanging indent
Line 540:21: W504 line break after binary operator
Line 547:36: W504 line break after binary operator
Line 552:21: E126 continuation line over-indented for hanging indent
Line 552:21: W504 line break after binary operator
Line 560:21: E126 continuation line over-indented for hanging indent
Line 560:21: W504 line break after binary operator
Line 567:36: W504 line break after binary operator
Line 585:23: E126 continuation line over-indented for hanging indent
Line 655:40: W504 line break after binary operator
Line 655:44: E126 continuation line over-indented for hanging indent
Line 734:21: E126 continuation line over-indented for hanging indent
Line 734:21: W504 line break after binary operator
Line 736:24: W504 line break after binary operator
Line 739:27: W504 line break after binary operator
Line 746:36: W504 line break after binary operator
Line 746:40: E126 continuation line over-indented for hanging indent
Line 751:21: E126 continuation line over-indented for hanging indent
Line 751:21: W504 line break after binary operator
Line 753:23: W504 line break after binary operator
Line 756:26: W504 line break after binary operator
Line 765:39: W504 line break after binary operator
Line 784:27: W503 line break before binary operator
Line 785:27: W503 line break before binary operator
Line 895:21: E126 continuation line over-indented for hanging indent
Line 895:21: W504 line break after binary operator
Line 903:21: E126 continuation line over-indented for hanging indent
Line 903:21: W504 line break after binary operator
Line 909:36: W504 line break after binary operator
Line 916:21: E126 continuation line over-indented for hanging indent
Line 916:21: W504 line break after binary operator
Line 924:21: E126 continuation line over-indented for hanging indent
Line 924:21: W504 line break after binary operator
Line 931:36: W504 line break after binary operator
Line 1069:21: E126 continuation line over-indented for hanging indent
Line 1069:21: W504 line break after binary operator
Line 1077:21: E126 continuation line over-indented for hanging indent
Line 1077:21: W504 line break after binary operator
Line 1084:36: W504 line break after binary operator
Line 1091:21: E126 continuation line over-indented for hanging indent
Line 1091:21: W504 line break after binary operator
Line 1099:21: E126 continuation line over-indented for hanging indent
Line 1099:21: W504 line break after binary operator
Line 1107:36: W504 line break after binary operator
Line 1254:17: E125 continuation line with same indent as next logical line
Line 1278:61: W504 line break after binary operator
Line 1279:61: W504 line break after binary operator
Line 1280:61: W504 line break after binary operator
Line 1408:30: E126 continuation line over-indented for hanging indent
Line 1481:30: E126 continuation line over-indented for hanging indent
Line 1498:32: W503 line break before binary operator
Line 1507:32: W504 line break after binary operator
Line 1529:17: E129 visually indented line with same indent as next logical line
Line 1647:26: E126 continuation line over-indented for hanging indent
Line 1882:21: E722 do not use bare 'except'
Line 1951:48: W504 line break after binary operator
Line 1954:48: W504 line break after binary operator
Line 1993:13: W504 line break after binary operator
Line 2006:59: W504 line break after binary operator
Line 2008:58: W504 line break after binary operator
Line 2040:59: W504 line break after binary operator
Line 2042:58: W504 line break after binary operator
Line 2079:59: W504 line break after binary operator
Line 2081:58: W504 line break after binary operator
Line 2114:25: W504 line break after binary operator
Line 2286:63: E712 comparison to False should be 'if cond is False:' or 'if not cond:'
Line 2362:27: W503 line break before binary operator
Line 2363:33: E126 continuation line over-indented for hanging indent
Line 2369:5: E266 too many leading '#' for block comment

Line 43:20: E126 continuation line over-indented for hanging indent
Line 45:20: E126 continuation line over-indented for hanging indent
Line 91:57: E126 continuation line over-indented for hanging indent

Line 54:13: E741 ambiguous variable name 'l'
Line 67:13: E741 ambiguous variable name 'l'
Line 80:13: E722 do not use bare 'except'
Line 83:13: E741 ambiguous variable name 'l'
Line 84:13: E741 ambiguous variable name 'l'
Line 85:13: E741 ambiguous variable name 'l'
Line 88:13: E741 ambiguous variable name 'l'
Line 92:13: E741 ambiguous variable name 'l'
Line 120:30: W504 line break after binary operator

Line 76:29: E261 at least two spaces before inline comment
Line 93:42: E225 missing whitespace around operator
Line 149:50: E225 missing whitespace around operator
Line 500:1: E266 too many leading '#' for block comment
Line 502:11: E225 missing whitespace around operator
Line 502:20: W291 trailing whitespace
Line 504:13: E225 missing whitespace around operator
Line 504:23: W291 trailing whitespace
Line 506:10: W291 trailing whitespace
Line 507:55: E225 missing whitespace around operator
Line 510:1: E305 expected 2 blank lines after class or function definition, found 1
Line 510:10: W605 invalid escape sequence '\w'
Line 510:14: W605 invalid escape sequence '\d'
Line 689:1: W391 blank line at end of file

Line 167:21: W504 line break after binary operator
Line 203:21: W504 line break after binary operator
Line 207:21: W504 line break after binary operator

Line 545:17: E741 ambiguous variable name 'l'
Line 584:28: W504 line break after binary operator
Line 587:28: W504 line break after binary operator

Comment last updated on November 16, 2018 at 17:17 Hours UTC

@coveralls
Copy link

coveralls commented Nov 16, 2018

Coverage Status

Coverage increased (+0.008%) to 48.229% when pulling d296f1d on samuelgarcia:fix_warnings_0.7 into de8b590 on NeuralEnsemble:master.

@apdavison apdavison merged commit 17f5473 into NeuralEnsemble:master Nov 16, 2018
@samuelgarcia samuelgarcia deleted the fix_warnings_0.7 branch April 10, 2019 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants