Closed adebahr closed 5 years ago
Hello @adebahr! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
apercal/modules/prepare.py
:Line 48:9: E265 block comment should start with '# ' Line 313:5: E303 too many blank lines (2) Line 324:201: E501 line too long (275 > 200 characters) Line 338:201: E501 line too long (272 > 200 characters) Line 352:201: E501 line too long (272 > 200 characters)
apercal/modules/split.py
:apercal/pipeline/start_pipeline.py
:Line 169:13: E265 block comment should start with '# ' Line 191:17: E265 block comment should start with '# ' Line 211:9: E265 block comment should start with '# ' Line 217:9: E265 block comment should start with '# '
Moved splitting into a separate module. Parallelisation was tested