pep8speaks
commented
5 years ago Hello @josecarlosgomezt! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:
- In the file
flame/apply.py:
Line 279:37: E127 continuation line over-indented for visual indent Line 286:22: E128 continuation line under-indented for visual indent Line 294:28: E211 whitespace before '(' Line 294:67: W291 trailing whitespace Line 295:1: W293 blank line contains whitespace Line 307:29: E127 continuation line over-indented for visual indent Line 312:29: E127 continuation line over-indented for visual indent Line 314:25: E128 continuation line under-indented for visual indent Line 317:1: W293 blank line contains whitespace Line 355:1: W293 blank line contains whitespace
- In the file
flame/learn.py:
Line 33:47: W291 trailing whitespace Line 34:49: W291 trailing whitespace Line 37:36: W291 trailing whitespace Line 71:27: E231 missing whitespace after ':' Line 71:40: E502 the backslash is redundant between brackets Line 72:34: E231 missing whitespace after ':' Line 72:54: E502 the backslash is redundant between brackets Line 73:28: E231 missing whitespace after ':' Line 76:39: E128 continuation line under-indented for visual indent Line 77:1: W293 blank line contains whitespace Line 79:40: W291 trailing whitespace Line 110:25: E117 over-indented Line 112:70: E231 missing whitespace after ','
- In the file
flame/stats/base_model.py:
Line 858:1: W293 blank line contains whitespace
- In the file
flame/stats/feature_selection.py:
Line 78:54: W291 trailing whitespace Line 96:33: W292 no newline at end of file
Migration of preprocessing to learn class.