Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JOSS-REVIEW: documentation #49

Closed
JannisHoch opened this issue Jun 4, 2021 · 6 comments
Closed

JOSS-REVIEW: documentation #49

JannisHoch opened this issue Jun 4, 2021 · 6 comments
Assignees

Comments

@JannisHoch
Copy link

hi @thurber,

i find the documentation overall good, but also pretty concise.
it would be great to include a jupyter notebook with an example application of your model.
this would not only give users an idea how a typical model workflow would look like, but also provide you with an opportunity to explain the model's functionality and set-up step-by-step and in greater detail.

i furthermore would like to see a 'statement of need' (similar as in the paper) in the documentation. not everyone is going to read the paper, but many will just go through the readme or rtd.

reference to: openjournals/joss-reviews#3221.

@JannisHoch
Copy link
Author

in addition to this:
the example application could be based on example data (if their size allows), which in turn could be used to provide a simple model which users can use to discover and play around with model functionalities.

@thurber thurber self-assigned this Jun 4, 2021
@thurber
Copy link
Contributor

thurber commented Jun 4, 2021

good idea, i'll add a jupyter notebook with the tutorial inside it

@cheginit
Copy link

cheginit commented Jun 4, 2021

@thurber Since you're working on the documentation, just wanted to remind you to update the installation instructions (using Python 3.7+)

@thurber
Copy link
Contributor

thurber commented Jun 7, 2021

I added a Jupyter notebook tutorial in pull request #50!

@thurber
Copy link
Contributor

thurber commented Jun 10, 2021

addressed by #50

@thurber thurber closed this as completed Jun 10, 2021
@JannisHoch
Copy link
Author

@thurber,

documentation in general is improved.
last thing i'd recommend is to link the notebook you have under notebooks to the readthedocs-documentation via the nblink extension. the package to install is nbconvert (https://nbconvert.readthedocs.io/en/latest/) and then link a ipynb.nblink file in index.rst file.
the nblink-file should include a relative path to the ipynb-file, looking like that:

{
    "path": "../../filename.ipynb"
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants