pep8speaks
commented
6 years ago Hello @sara-bah! Thanks for submitting the PR.
- In the file
gravityspy/ML/GS_utils.py, following are the PEP8 issues :
Line 25:1: E402 module level import not at top of file Line 26:1: E402 module level import not at top of file Line 27:1: E402 module level import not at top of file Line 28:1: E402 module level import not at top of file Line 29:1: E402 module level import not at top of file Line 30:1: E402 module level import not at top of file Line 31:1: E402 module level import not at top of file Line 32:1: E402 module level import not at top of file Line 33:1: E402 module level import not at top of file Line 34:1: E402 module level import not at top of file Line 35:1: E402 module level import not at top of file Line 35:14: E401 multiple imports on one line Line 36:1: E402 module level import not at top of file Line 37:1: E402 module level import not at top of file Line 38: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 41: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 44:1: E402 module level import not at top of file Line 45:1: E402 module level import not at top of file Line 48:1: E402 module level import not at top of file Line 49:1: E402 module level import not at top of file Line 50:1: E402 module level import not at top of file Line 53:1: E265 block comment should start with '# ' Line 54:45: E231 missing whitespace after ',' Line 54:80: E501 line too long (81 > 79 characters) Line 61:80: E501 line too long (105 > 79 characters) Line 66:80: E501 line too long (106 > 79 characters) Line 68:38: E231 missing whitespace after ',' Line 72:1: E265 block comment should start with '# ' Line 75:10: E211 whitespace before '(' Line 80:14: E211 whitespace before '(' Line 84:9: E265 block comment should start with '# ' Line 89:1: E303 too many blank lines (3) Line 93:9: E722 do not use bare except' Line 96:46: E202 whitespace before ')' Line 99:80: E501 line too long (140 > 79 characters) Line 102:1: E302 expected 2 blank lines, found 1 Line 102:45: E231 missing whitespace after ',' Line 106:9: E722 do not use bare except' Line 109:14: E211 whitespace before '(' Line 110:5: E265 block comment should start with '# ' Line 111:5: E265 block comment should start with '# ' Line 114:1: E302 expected 2 blank lines, found 1 Line 115:5: E265 block comment should start with '# ' Line 117:5: E265 block comment should start with '# ' Line 118:5: E265 block comment should start with '# ' Line 119:5: E265 block comment should start with '# ' Line 121:5: E265 block comment should start with '# ' Line 121:80: E501 line too long (105 > 79 characters) Line 126:5: E266 too many leading '#' for block comment Line 141:5: E266 too many leading '#' for block comment Line 146:40: E225 missing whitespace around operator Line 146:80: E501 line too long (102 > 79 characters) Line 147:5: E265 block comment should start with '# ' Line 152:72: E202 whitespace before ')' Line 159:80: E501 line too long (104 > 79 characters) Line 164:5: E266 too many leading '#' for block comment Line 177:80: E501 line too long (112 > 79 characters) Line 179:5: E266 too many leading '#' for block comment Line 181:80: E501 line too long (127 > 79 characters) Line 185:40: E225 missing whitespace around operator Line 185:80: E501 line too long (102 > 79 characters) Line 186:5: E265 block comment should start with '# ' Line 194:59: E202 whitespace before ']' Line 202:7: E111 indentation is not a multiple of four Line 203:7: E114 indentation is not a multiple of four (comment) Line 203:7: E265 block comment should start with '# ' Line 204:7: E111 indentation is not a multiple of four Line 208:9: E265 block comment should start with '# ' Line 208:80: E501 line too long (122 > 79 characters) Line 213:16: E114 indentation is not a multiple of four (comment) Line 217:17: E265 block comment should start with '# ' Line 222:16: E114 indentation is not a multiple of four (comment) Line 226:1: E302 expected 2 blank lines, found 1 Line 236:80: E501 line too long (123 > 79 characters) Line 244:80: E501 line too long (127 > 79 characters) Line 262:1: E302 expected 2 blank lines, found 1 Line 266:9: E722 do not use bare except' Line 272:80: E501 line too long (93 > 79 characters) Line 275:1: E265 block comment should start with '# ' Line 276:1: E302 expected 2 blank lines, found 1 Line 281:80: E501 line too long (117 > 79 characters) Line 287:5: E303 too many blank lines (2) Line 287:80: E501 line too long (80 > 79 characters) Line 292:80: E501 line too long (80 > 79 characters) Line 297:80: E501 line too long (80 > 79 characters) Line 307:10: E211 whitespace before '(' Line 310:1: E305 expected 2 blank lines after class or function definition, found 1 Line 318:65: W291 trailing whitespace Line 331:65: W291 trailing whitespace Line 348:5: E265 block comment should start with '# ' Line 349:5: E265 block comment should start with '# ' Line 354:29: E127 continuation line over-indented for visual indent Line 355:29: E127 continuation line over-indented for visual indent Line 356:29: E127 continuation line over-indented for visual indent Line 359:5: E265 block comment should start with '# ' Line 361:5: E265 block comment should start with '# ' Line 379:5: E265 block comment should start with '# ' Line 384:29: E127 continuation line over-indented for visual indent Line 385:29: E127 continuation line over-indented for visual indent Line 386:29: E127 continuation line over-indented for visual indent Line 391:5: E265 block comment should start with '# ' Line 395:29: E127 continuation line over-indented for visual indent Line 396:29: E127 continuation line over-indented for visual indent Line 399:5: E265 block comment should start with '# ' Line 408:1: W391 blank line at end of file
- In the file
gravityspy/ML/train_classifier.py, following are the PEP8 issues :
Line 1:80: E501 line too long (87 > 79 characters) Line 7:1: E265 block comment should start with '# ' Line 17:1: E302 expected 2 blank lines, found 1 Line 17:80: E501 line too long (81 > 79 characters) Line 31:80: E501 line too long (82 > 79 characters) Line 34:80: E501 line too long (82 > 79 characters) Line 37:80: E501 line too long (82 > 79 characters) Line 40:80: E501 line too long (82 > 79 characters) Line 70:5: E303 too many blank lines (2) Line 90:66: E231 missing whitespace after ',' Line 90:80: E501 line too long (117 > 79 characters) Line 91:80: E501 line too long (112 > 79 characters) Line 91:91: E231 missing whitespace after ',' Line 92:80: E501 line too long (117 > 79 characters) Line 92:96: E231 missing whitespace after ',' Line 100:19: E128 continuation line under-indented for visual indent Line 101:19: E128 continuation line under-indented for visual indent Line 105:80: E501 line too long (96 > 79 characters) Line 106:80: E501 line too long (103 > 79 characters) Line 107:80: E501 line too long (93 > 79 characters) Line 108:80: E501 line too long (102 > 79 characters) Line 112:9: E701 multiple statements on one line (colon) Line 126:80: E501 line too long (81 > 79 characters) Line 127:80: E501 line too long (86 > 79 characters) Line 128:80: E501 line too long (92 > 79 characters) Line 129:80: E501 line too long (93 > 79 characters) Line 133:80: E501 line too long (95 > 79 characters) Line 141:80: E501 line too long (81 > 79 characters) Line 144:80: E501 line too long (81 > 79 characters) Line 154:80: E501 line too long (81 > 79 characters) Line 156:80: E501 line too long (81 > 79 characters) Line 163:80: E501 line too long (81 > 79 characters) Line 165:80: E501 line too long (81 > 79 characters) Line 170:4: E111 indentation is not a multiple of four Line 171:4: E111 indentation is not a multiple of four Line 172:4: E111 indentation is not a multiple of four Line 173:4: E111 indentation is not a multiple of four Line 175:4: E111 indentation is not a multiple of four Line 176:4: E111 indentation is not a multiple of four Line 178:4: E114 indentation is not a multiple of four (comment) Line 178:80: E501 line too long (104 > 79 characters) Line 179:4: E111 indentation is not a multiple of four Line 181:4: E114 indentation is not a multiple of four (comment) Line 182:4: E111 indentation is not a multiple of four Line 182:80: E501 line too long (126 > 79 characters) Line 184:4: E114 indentation is not a multiple of four (comment) Line 185:4: E111 indentation is not a multiple of four Line 186:4: E111 indentation is not a multiple of four Line 187:4: E111 indentation is not a multiple of four Line 188:4: E111 indentation is not a multiple of four Line 190:4: E111 indentation is not a multiple of four Line 190:80: E501 line too long (80 > 79 characters) Line 192:4: E111 indentation is not a multiple of four Line 193:1: W391 blank line at end of file
coveralls
updating train_clasifier.py and GS_utils deleting function_fusion file as only one function is needed from this file and I moved that function to GS_utils under the name "concatenate_views"