-
Notifications
You must be signed in to change notification settings - Fork 5
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
Revise docs #119
Comments
Tasks can be collected here (jnnr): @monika-o please do another review of the docs to find out the parts that need to be adressed before the release.
|
The above list has got quite extensive. In my opinion, the most important tasks are the revision of the model pipeline, simply because it looks very incomplete, and a documentation of what @SabineHaas wrote in #81. |
Some docstrings are not rendered with |
There are two links in the installation section where the text that should clickable as link is displayed in square brackets and the actual link (which should be hidden) follows in normal brackets. This concerns pandoc installation and oemof.solph. |
Thanks for specifying @monika-o. Looks like this was supposed to be a feature and not a bug ;-) However, I fixed this with commit 506d180 in PR #263. |
Time has passed and I have a few new comments: |
Thanks for looking into it @monika-o
Could you write an issue for this? It has to be checked if the snakemake execution works with windows by now.
Thanks I took it out.
I fixed this with commit 13ea89f. However download_resources.py needs a docstring, that can be displayed in the model pipeline (I just added download_resources there because it was missing). Could you write an issue for this too? |
@Stefanie08 I think we've already looked at this but could you check if we have this ↑ covered? Basic message should be:
Actually, this should be documented in |
Could you do this in PR #327 as well @Stefanie08? I think it should not take too much time. Thank you! |
The docs are partly outdated and need to be revised.
Open docs issues:
The text was updated successfully, but these errors were encountered: