Improving documentation (JOSS review) about deeplenstronomy

[rmorgan10]

Links in the "Getting Started" Notebook are now fixed by commit 68ffd33

[shreyasbapat]

I won't create a new issue for the test suite since it is already mentioned here :D

[shreyasbapat]

I would also like to point out that even though the package has excellent docstrings pretty much everywhere, there's no simple way to see what the API looks like. I feel there is a need to have the API Documentation for the same. Which can be easily achieved using sphinx.

[rmorgan10]

I've created a docs page to make the docstrings more accessible with commit a960561

https://deepskies.github.io/deeplenstronomy/docs

[rmorgan10]

I've linked the docs page in the main readme so that users can easily find it with commit 7d1d0b7

[rmorgan10]

@jiwoncpark I'm planning on addressing your comments about the tests in @shreyasbapat 's issue #42 . When you get a chance, could you look over the improvements I've made to the documentation and let me know if you feel like this issue can be closed?

[rmorgan10]

@jiwoncpark I've also updated the paper text based on your recommendations with the following commits:

• d03bc5e
• 0533893
• 0593e7e

[pdebuyl]

The modules distributions and surveys don't have docstrings, it would be good to add them.

[rmorgan10]

The modules distributions and surveys don't have docstrings, it would be good to add them.

@pdebuyl The reason I left docstrings out for those modules is they are internal functions, never called by the user. The only reason I included them in the documentation and didn't make them private methods was to provide a quickly accessible list of all possible surveys and distributions.

Given that information, do you think the docstrings are still necessary for those modules?

[pdebuyl]

Given that information, do you think the docstrings are still necessary for those modules?

Yes. But a minimal docstring is fine. Else, the only starting point to figure what a function does is its name or the source code, which is as good as not providing their listing.

[rmorgan10]

@pdebuyl I've added docstrings to the distributions.py, surveys.py, and check.py modules with commit e906f49 and I've regenerated all the docs pages.

[rmorgan10]

@jiwoncpark Can I go ahead and close this issue?

[jiwoncpark]

LGTM! Yes, please go ahead.

[rmorgan10]

Great! Thanks for the review!