apertif / apercal

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

- updated default.cfg with polarisation parameters #246

Closed adebahr closed 5 years ago

adebahr commented 5 years ago
pep8speaks commented 5 years ago

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

Line 135:1: W293 blank line contains whitespace Line 788:1: W293 blank line contains whitespace

Line 121:1: E302 expected 2 blank lines, found 1

Comment last updated at 2019-06-28 21:19:27 UTC
rs1701 commented 5 years ago

I ran several tests of the module in the pipeline code and seems stable. I think that it can be merged.