pep8speaks
commented
3 months ago Hello @mperrin, Thank you for updating !
- In the file
webbpsf/tests/test_nircam.py:
Line 26:1: E302 expected 2 blank lines, found 1 Line 29:1: E302 expected 2 blank lines, found 1 Line 32:1: E302 expected 2 blank lines, found 1 Line 35:1: E302 expected 2 blank lines, found 1 Line 36:11: E271 multiple spaces after keyword Line 37:17: E126 continuation line over-indented for hanging indent Line 38:13: E126 continuation line over-indented for hanging indent Line 40:1: E302 expected 2 blank lines, found 1 Line 43:1: E302 expected 2 blank lines, found 1 Line 57:1: E302 expected 2 blank lines, found 1 Line 192:126: E501 line too long (126 > 125 characters) Line 321:126: E501 line too long (142 > 125 characters)
Add a unit test for the DHS support added in #845
This tests that we can select one of the DHS sub apertures as a pupil mask, make a PSF, and that the resulting PSF has only a couple percent of the total intensity (as expected, given the small size of these sub apertures)