search

spacetelescope / jwst_coronagraph_visibility

Visualize approximate pointing constraints for JWST coronagraphs.
https://github.com/spacetelescope/jwst_visibility/blob/master/docs/index.rst
BSD 3-Clause "New" or "Revised" License
8 stars 10 forks source link

Updating docstrings #48

Closed brynickson closed 3 years ago

brynickson commented 3 years ago
  1. Added docstrings to all the code.
  2. Changed version in setup.py
pep8speaks commented 3 years ago

Hello @brynickson, Thank you for submitting the Pull Request !

Line 22:1: E402 module level import not at top of file Line 23:1: E402 module level import not at top of file Line 25:1: E402 module level import not at top of file Line 26:1: E402 module level import not at top of file Line 27:1: E402 module level import not at top of file Line 28:1: E402 module level import not at top of file Line 29:1: E402 module level import not at top of file Line 30:1: E402 module level import not at top of file Line 31:1: E402 module level import not at top of file Line 32:1: E402 module level import not at top of file Line 42:1: E402 module level import not at top of file Line 43:1: E402 module level import not at top of file Line 44:1: E402 module level import not at top of file Line 100:8: E126 continuation line over-indented for hanging indent

Line 71:34: W504 line break after binary operator Line 72:60: W504 line break after binary operator Line 100:52: W504 line break after binary operator Line 189:51: W504 line break after binary operator Line 503:24: W504 line break after binary operator Line 504:26: W504 line break after binary operator Line 511:25: W504 line break after binary operator Line 512:25: W504 line break after binary operator

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

Line 14:1: E302 expected 2 blank lines, found 1 Line 67:1: E302 expected 2 blank lines, found 1 Line 122:1: E302 expected 2 blank lines, found 1 Line 149:1: E302 expected 2 blank lines, found 1 Line 189:5: E303 too many blank lines (2) Line 189:31: E261 at least two spaces before inline comment Line 206:29: E241 multiple spaces after ',' Line 207:29: E241 multiple spaces after ',' Line 208:29: E241 multiple spaces after ',' Line 209:29: E241 multiple spaces after ',' Line 211:1: E302 expected 2 blank lines, found 1 Line 232:1: E302 expected 2 blank lines, found 1 Line 253:1: E302 expected 2 blank lines, found 1 Line 274:1: E302 expected 2 blank lines, found 1

If you have not done so, please consult the jwql Style Guide