Closed manuelpastor closed 2 years ago
Hello @manuelpastor! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:
flame/apply.py
Line 68:5: E303 too many blank lines (2) Line 68:20: E211 whitespace before '(' Line 74:36: E211 whitespace before '(' Line 75:1: W293 blank line contains whitespace Line 80:40: W291 trailing whitespace Line 215:80: E501 line too long (92 > 79 characters)
flame/build.py
Line 80:31: E211 whitespace before '(' Line 84:29: E211 whitespace before '(' Line 87:25: E211 whitespace before '(' Line 88:17: E128 continuation line under-indented for visual indent Line 88:80: E501 line too long (103 > 79 characters) Line 92:25: E211 whitespace before '(' Line 92:80: E501 line too long (80 > 79 characters) Line 93:17: E128 continuation line under-indented for visual indent Line 93:80: E501 line too long (103 > 79 characters) Line 94:1: W293 blank line contains whitespace Line 97:21: E211 whitespace before '(' Line 97:80: E501 line too long (137 > 79 characters) Line 100:44: E211 whitespace before '(' Line 101:45: E128 continuation line under-indented for visual indent Line 103:22: E211 whitespace before '('
flame/learn.py
Line 81:20: E211 whitespace before '(' Line 85:14: E225 missing whitespace around operator Line 93:25: W291 trailing whitespace Line 99:39: E211 whitespace before '(' Line 99:80: E501 line too long (88 > 79 characters) Line 101:22: E211 whitespace before '(' Line 281:14: W291 trailing whitespace Line 292:80: E501 line too long (120 > 79 characters) Line 293:80: E501 line too long (111 > 79 characters) Line 458:46: E211 whitespace before '(' Line 458:80: E501 line too long (106 > 79 characters) Line 461:41: E225 missing whitespace around operator Line 463:26: E211 whitespace before '('
flame/manage.py
Line 258:1: W293 blank line contains whitespace Line 292:8: E713 test for membership should be 'not in' Line 293:1: W293 blank line contains whitespace Line 302:55: E231 missing whitespace after ',' Line 302:80: E501 line too long (81 > 79 characters) Line 303:1: W293 blank line contains whitespace Line 311:80: E501 line too long (85 > 79 characters) Line 316:25: E211 whitespace before '(' Line 320:40: E211 whitespace before '(' Line 323:34: E222 multiple spaces after operator Line 323:44: E231 missing whitespace after ',' Line 323:47: E231 missing whitespace after ',' Line 323:64: W291 trailing whitespace Line 324:45: E231 missing whitespace after ',' Line 324:48: E231 missing whitespace after ',' Line 324:66: W291 trailing whitespace Line 325:45: E231 missing whitespace after ',' Line 325:48: E231 missing whitespace after ',' Line 327:43: E231 missing whitespace after ',' Line 327:46: E231 missing whitespace after ',' Line 327:61: W291 trailing whitespace Line 328:44: E231 missing whitespace after ',' Line 328:47: E231 missing whitespace after ',' Line 329:46: E231 missing whitespace after ',' Line 329:49: E231 missing whitespace after ',' Line 330:46: E231 missing whitespace after ',' Line 330:49: E231 missing whitespace after ',' Line 331:50: E231 missing whitespace after ',' Line 331:53: E231 missing whitespace after ',' Line 332:50: E231 missing whitespace after ',' Line 332:53: E231 missing whitespace after ',' Line 334:30: E222 multiple spaces after operator Line 334:48: E231 missing whitespace after ',' Line 334:51: E231 missing whitespace after ',' Line 334:76: W291 trailing whitespace Line 335:45: E231 missing whitespace after ',' Line 335:48: E231 missing whitespace after ',' Line 335:70: W291 trailing whitespace Line 336:56: E231 missing whitespace after ',' Line 336:59: E231 missing whitespace after ',' Line 336:80: E501 line too long (91 > 79 characters) Line 337:44: E231 missing whitespace after ',' Line 337:47: E231 missing whitespace after ',' Line 337:55: W291 trailing whitespace Line 338:50: E231 missing whitespace after ',' Line 338:53: E231 missing whitespace after ',' Line 338:58: W291 trailing whitespace Line 339:53: E231 missing whitespace after ',' Line 339:56: E231 missing whitespace after ',' Line 339:61: W291 trailing whitespace Line 340:48: E231 missing whitespace after ',' Line 340:51: E231 missing whitespace after ',' Line 340:58: W291 trailing whitespace Line 341:42: E231 missing whitespace after ',' Line 341:45: E231 missing whitespace after ',' Line 341:52: W291 trailing whitespace Line 349:28: E211 whitespace before '(' Line 350:21: E128 continuation line under-indented for visual indent Line 351:21: E128 continuation line under-indented for visual indent Line 352:21: E128 continuation line under-indented for visual indent Line 353:21: E128 continuation line under-indented for visual indent Line 354:21: E128 continuation line under-indented for visual indent Line 355:28: E211 whitespace before '(' Line 356:21: E128 continuation line under-indented for visual indent Line 357:21: E128 continuation line under-indented for visual indent Line 358:21: E128 continuation line under-indented for visual indent Line 359:21: E128 continuation line under-indented for visual indent Line 360:21: E128 continuation line under-indented for visual indent Line 361:28: E211 whitespace before '(' Line 362:21: E128 continuation line under-indented for visual indent Line 363:21: E128 continuation line under-indented for visual indent Line 364:21: E128 continuation line under-indented for visual indent Line 365:21: E128 continuation line under-indented for visual indent Line 366:21: E128 continuation line under-indented for visual indent Line 368:58: E231 missing whitespace after ',' Line 370:30: E211 whitespace before '(' Line 374:17: E211 whitespace before '(' Line 377:80: E501 line too long (91 > 79 characters) Line 379:1: W293 blank line contains whitespace Line 412:80: E501 line too long (84 > 79 characters) Line 412:85: W291 trailing whitespace Line 431:51: E231 missing whitespace after ',' Line 434:49: E231 missing whitespace after ',' Line 438:41: E211 whitespace before '(' Line 439:38: E211 whitespace before '(' Line 446:51: E231 missing whitespace after ',' Line 469:80: E501 line too long (106 > 79 characters) Line 471:22: E225 missing whitespace around operator Line 471:23: E203 whitespace before ':' Line 472:80: E501 line too long (89 > 79 characters) Line 474:80: E501 line too long (93 > 79 characters) Line 526:1: W293 blank line contains whitespace Line 527:33: E211 whitespace before '(' Line 527:71: E202 whitespace before ')' Line 530:76: W291 trailing whitespace Line 531:13: E128 continuation line under-indented for visual indent Line 531:80: E501 line too long (84 > 79 characters) Line 531:85: W291 trailing whitespace Line 532:17: W291 trailing whitespace Line 534:44: E211 whitespace before '(' Line 535:13: E722 do not use bare 'except' Line 537:36: E231 missing whitespace after ',' Line 541:25: E225 missing whitespace around operator Line 541:61: E202 whitespace before ')'
flame/odata.py
Line 247:80: E501 line too long (82 > 79 characters)
flame/parameters.py
Line 415:80: E501 line too long (109 > 79 characters) Line 478:65: E225 missing whitespace around operator Line 479:50: E222 multiple spaces after operator Line 481:50: E222 multiple spaces after operator Line 508:47: E225 missing whitespace around operator Line 509:33: E222 multiple spaces after operator Line 511:33: E222 multiple spaces after operator
flame/stats/GNB.py
Line 70:80: E501 line too long (91 > 79 characters) Line 70:84: E711 comparison to None should be 'if cond is not None:' Line 72:80: E501 line too long (91 > 79 characters) Line 72:84: E711 comparison to None should be 'if cond is not None:' Line 73:57: W291 trailing whitespace Line 74:1: W293 blank line contains whitespace Line 77:80: E501 line too long (88 > 79 characters) Line 79:21: E225 missing whitespace around operator Line 79:40: E225 missing whitespace around operator Line 81:80: E501 line too long (107 > 79 characters)
flame/stats/PLSR.py
Line 66:17: E211 whitespace before '(' Line 71:36: E211 whitespace before '(' Line 73:42: W291 trailing whitespace Line 125:1: W293 blank line contains whitespace Line 162:29: E211 whitespace before '(' Line 175:80: E501 line too long (86 > 79 characters) Line 179:80: E501 line too long (80 > 79 characters) Line 180:43: E211 whitespace before '(' Line 180:80: E501 line too long (87 > 79 characters) Line 182:26: E211 whitespace before '(' Line 213:42: W292 no newline at end of file Line 213:42: W292 no newline at end of file Line 213:42: W292 no newline at end of file
flame/stats/base_model.py
Line 337:80: E501 line too long (98 > 79 characters) Line 338:14: E231 missing whitespace after ',' Line 340:35: E231 missing whitespace after ',' Line 340:41: E231 missing whitespace after ',' Line 698:80: E501 line too long (102 > 79 characters) Line 699:18: E231 missing whitespace after ',' Line 701:39: E231 missing whitespace after ',' Line 701:45: E231 missing whitespace after ',' Line 823:80: E501 line too long (80 > 79 characters) Line 824:43: E211 whitespace before '(' Line 824:60: E231 missing whitespace after ',' Line 824:80: E501 line too long (86 > 79 characters) Line 826:14: W291 trailing whitespace Line 995:59: E231 missing whitespace after ',' Line 996:1: W293 blank line contains whitespace
flame/util/utils.py
Line 233:9: E722 do not use bare 'except' Line 238:36: E211 whitespace before '('
Hello @manuelpastor! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:
flame/apply.py:flame/build.py:flame/learn.py:flame/manage.py:flame/odata.py:flame/parameters.py:flame/stats/GNB.py:flame/stats/PLSR.py:flame/stats/base_model.py:flame/util/utils.py: