search

apertif / apercal

GNU General Public License v3.0
3 stars 9 forks source link

Preflag complex flag #266

Closed rs1701 closed 5 years ago

rs1701 commented 5 years ago

Added feature that allows the user to provide a list of casa flagging commands in a single file. The flagging commands (for different beams) are provided by a single json-file. A template has been added, too. Also, the setinitdir function now uses os.path.join to combine the different (sub-)directories

pep8speaks commented 5 years ago

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

Line 526:1: W293 blank line contains whitespace Line 549:1: W293 blank line contains whitespace Line 557:1: W293 blank line contains whitespace Line 567:1: W293 blank line contains whitespace Line 570:17: E265 block comment should start with '# ' Line 573:17: E303 too many blank lines (2) Line 574:46: W601 .has_key() is deprecated, use 'in' Line 582:33: E272 multiple spaces before keyword Line 585:1: W293 blank line contains whitespace Line 597:1: W293 blank line contains whitespace Line 602:33: E265 block comment should start with '# ' Line 604:33: E265 block comment should start with '# ' Line 613:33: E265 block comment should start with '# ' Line 621:1: W293 blank line contains whitespace Line 624:33: E265 block comment should start with '# ' Line 630:73: E225 missing whitespace around operator

Comment last updated at 2019-09-19 12:49:11 UTC
rs1701 commented 5 years ago

Removed the change to setinitdir