phi-grib / flame

Modeling framework for eTRANSAFE project
GNU General Public License v3.0
12 stars 10 forks source link

Smarts similarity #203

Closed manuelpastor closed 3 years ago

pep8speaks commented 3 years ago

Hello @manuelpastor! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 100:39: E202 whitespace before ')' Line 151:80: E501 line too long (82 > 79 characters) Line 151:81: E203 whitespace before ':' Line 164:18: E128 continuation line under-indented for visual indent

Line 964:20: E211 whitespace before '(' Line 966:29: E128 continuation line under-indented for visual indent Line 969:29: E128 continuation line under-indented for visual indent Line 969:80: E501 line too long (89 > 79 characters) Line 972:80: E501 line too long (92 > 79 characters) Line 1345:37: E231 missing whitespace after ',' Line 1361:1: W293 blank line contains whitespace

Line 88:15: E211 whitespace before '(' Line 89:12: E713 test for membership should be 'not in' Line 107:34: E231 missing whitespace after ',' Line 111:36: E231 missing whitespace after ',' Line 169:43: E211 whitespace before '('

Line 47:1: W293 blank line contains whitespace Line 71:52: E225 missing whitespace around operator Line 73:1: W293 blank line contains whitespace Line 75:80: E501 line too long (94 > 79 characters) Line 77:80: E501 line too long (185 > 79 characters) Line 78:63: E231 missing whitespace after ',' Line 78:80: E501 line too long (81 > 79 characters) Line 224:14: E114 indentation is not a multiple of four (comment) Line 224:14: E116 unexpected indentation (comment) Line 233:30: E225 missing whitespace around operator Line 242:80: E501 line too long (109 > 79 characters) Line 244:19: E231 missing whitespace after ',' Line 250:1: W293 blank line contains whitespace Line 251:21: E211 whitespace before '(' Line 386:11: E271 multiple spaces after keyword

Line 316:1: W293 blank line contains whitespace