pep8speaks
commented
5 years ago Hello @mikkokotila! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
- In the file
talos/__init__.py:
Line 6:1: E402 module level import not at top of file Line 7:1: E402 module level import not at top of file Line 16:1: E402 module level import not at top of file
- In the file
talos/autom8/autoparams.py:
Line 200:80: E501 line too long (81 > 79 characters)
- In the file
talos/autom8/autopredict.py:
Line 1:1: E302 expected 2 blank lines, found 0 Line 100:80: E501 line too long (84 > 79 characters) Line 103:80: E501 line too long (82 > 79 characters)
- In the file
talos/commands/analyze.py:
Line 183:80: E501 line too long (81 > 79 characters) Line 205:80: E501 line too long (80 > 79 characters) Line 220:80: E501 line too long (82 > 79 characters)
- In the file
talos/model/early_stopper.py:
Line 16:80: E501 line too long (83 > 79 characters)
- In the file
talos/reducers/reduce_run.py:
Line 31:19: E712 comparison to True should be 'if cond is True:' or 'if cond:'
- In the file
talos/reducers/reduce_utils.py:
Line 29:80: E501 line too long (87 > 79 characters)
- In the file
talos/scan/scan_addon.py:
Line 53:80: E501 line too long (80 > 79 characters)
- In the file
talos/templates/datasets.py:
Line 21:80: E501 line too long (127 > 79 characters)
- In the file
test/commands/test_latest.py:
Line 36:80: E501 line too long (94 > 79 characters)
- In the file
test/commands/test_scan.py:
Line 66:80: E501 line too long (97 > 79 characters) Line 151:5: E303 too many blank lines (2)
See individual commits for more details.