rdemaria / pysixtrack

Reference sixtrack tracking engine in pure python
Apache License 2.0
1 stars 12 forks source link

Beam beam cleanup #8

Closed giadarol closed 5 years ago

giadarol commented 5 years ago
pep8speaks commented 5 years ago

Hello @giadarol! Thanks for submitting the PR.

giadarol commented 5 years ago

I propose to merge as it is (to better visualise the changes). Immediately after the merge I can autopep8 the code and make a separate pull request

Hello @giadarol! Thanks for submitting the PR.

Line 17:5: E265 block comment should start with '# ' Line 40:80: E501 line too long (81 > 79 characters) Line 41:5: E265 block comment should start with '# ' Line 41:80: E501 line too long (94 > 79 characters) Line 59:80: E501 line too long (91 > 79 characters) Line 72:80: E501 line too long (88 > 79 characters) Line 73:80: E501 line too long (97 > 79 characters) Line 76:80: E501 line too long (85 > 79 characters) Line 77:80: E501 line too long (117 > 79 characters) Line 78:80: E501 line too long (107 > 79 characters) Line 91:80: E501 line too long (86 > 79 characters) Line 92:80: E501 line too long (87 > 79 characters) Line 115:5: E265 block comment should start with '# '

Line 164:52: W291 trailing whitespace Line 165:17: E127 continuation line over-indented for visual indent Line 165:44: W291 trailing whitespace Line 166:17: E127 continuation line over-indented for visual indent Line 166:46: W291 trailing whitespace Line 167:17: E127 continuation line over-indented for visual indent Line 167:46: W291 trailing whitespace Line 168:17: E127 continuation line over-indented for visual indent Line 168:56: W291 trailing whitespace Line 169:17: E127 continuation line over-indented for visual indent Line 169:62: W291 trailing whitespace Line 170:17: E127 continuation line over-indented for visual indent Line 170:52: W291 trailing whitespace Line 171:17: E127 continuation line over-indented for visual indent Line 171:74: W291 trailing whitespace Line 172:17: E127 continuation line over-indented for visual indent Line 192:80: E501 line too long (127 > 79 characters) Line 194:80: E501 line too long (144 > 79 characters) Line 195:80: E501 line too long (194 > 79 characters) Line 195:114: E128 continuation line under-indented for visual indent Line 200:80: E501 line too long (128 > 79 characters) Line 201:80: E501 line too long (89 > 79 characters) Line 202:80: E501 line too long (83 > 79 characters)

Line 222:80: E501 line too long (89 > 79 characters) Line 242:80: E501 line too long (92 > 79 characters) Line 292:79: W291 trailing whitespace Line 293:54: W291 trailing whitespace Line 294:61: W291 trailing whitespace Line 295:61: W291 trailing whitespace Line 296:76: W291 trailing whitespace Line 297:80: E501 line too long (91 > 79 characters) Line 297:92: W291 trailing whitespace Line 298:62: W291 trailing whitespace Line 299:80: E501 line too long (103 > 79 characters) Line 299:104: W291 trailing whitespace Line 301:80: E501 line too long (80 > 79 characters) Line 302:80: E501 line too long (89 > 79 characters)

rdemaria commented 5 years ago

Ok, for this time ;-)