Closed rs1701 closed 4 years ago
Hello @rs1701! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
apercal/modules/mosaic_v2.py
:Line 1503:131: E226 missing whitespace around arithmetic operator Line 1503:201: E501 line too long (253 > 200 characters) Line 1506:56: E226 missing whitespace around arithmetic operator Line 1506:61: E226 missing whitespace around arithmetic operator Line 1509:157: E226 missing whitespace around arithmetic operator Line 1518:52: E226 missing whitespace around arithmetic operator Line 1523:127: E226 missing whitespace around arithmetic operator Line 1525:122: E226 missing whitespace around arithmetic operator Line 1529:48: E226 missing whitespace around arithmetic operator Line 1579:35: E226 missing whitespace around arithmetic operator
scripts/mosaic/run_mosaic_v2_continuum.py
:Line 29:1: E265 block comment should start with '# ' Line 30:1: E265 block comment should start with '# ' Line 38:201: E501 line too long (228 > 200 characters) Line 177:201: E501 line too long (343 > 200 characters)
scripts/mosaic/run_mosaic_v2_continuum_template.py
:Line 10:23: W291 trailing whitespace Line 11:69: W291 trailing whitespace Line 93:57: E226 missing whitespace around arithmetic operator
Added new mosaic module including script with default settings