Closed rs1701 closed 5 years ago
Hello @rs1701! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:
apercal/modules/line.py
:Line 35:45: E262 inline comment should start with '# ' Line 125:95: W291 trailing whitespace Line 127:121: E226 missing whitespace around arithmetic operator Line 148:1: W293 blank line contains whitespace Line 155:117: E226 missing whitespace around arithmetic operator Line 170:33: E201 whitespace after '(' Line 170:98: E202 whitespace before ')' Line 181:33: E201 whitespace after '(' Line 189:41: E225 missing whitespace around operator Line 189:45: W291 trailing whitespace Line 256:1: W293 blank line contains whitespace Line 259:1: W293 blank line contains whitespace Line 421:77: E231 missing whitespace after ',' Line 945:96: W291 trailing whitespace Line 951:1: W293 blank line contains whitespace
Updated line module to survey specification and parallelised line step in pipeline (temprorarily). The specifications of the subband cubes can be changed in the config file. Further testing of the stability and performance of the module is necessary. Tests have shown that it is working. The module should be able to handle failing on a subband cube and continue with the next one. In case line fails, it should also delete auxiliary files by default (can be turned off in the config file)