-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add Github Pages Documentation #63
Conversation
Guys, this is as much as I can get done for today (I might have time to do a little more later maybe). I spent a long time making the method of collecting the source code robust. Also, there are added instructions for the Sphinx installation so that we can autodoc the docstrings (which need more work!). Please feel free to add to this as you see fit. |
Mat when I go to pull this PR I am getting an error. I believe it is because this is essentially a separate repository. Typically I would:
However, in this case, I get the following error. I am considering cloning from your fork. Do you have a more correct solution? |
Try git fetch --all Assuming h0r5e is the name of the remote to my fork. |
Same error as the first screenshot. It seems to be working for Ryan. |
Don't try to create a branch called gh-pages because it already exists. |
You wanna try and fix this quick on hangouts? |
…b packages and main functions
@ryancoe how are you feeling about this one now? It might be worth pulling this and spending some time going to fix the inline documentation of the WecOptTool package. What do you think? |
Well, I guess we can't pull it, but at least park it for now. |
I think it's ready to go. The two outstanding items in my mind are
|
I think having the API for the WecOptTool package is useful as it gives the scope of the options available to the user at this point. I'll start updating the doc strings and we'll see how it's going by our next meeting. Unless that is you want to make it public now, in which case I agree we should take it out. I'll need admin access on the repo to set it up, whenever we publish, if that's ok? Happy to relinquish afterwards. Also I think your licence says MHkit... |
user/example.rst
Outdated
|
||
.. literalinclude:: /git_src/example.m | ||
:language: matlab | ||
:lines: 42-46 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line 42 of example.m has optimization misspelled.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll fix this in relation to #71. The line numbers for this whole page will need modified because there is now a GPL header at the top of the file.
Because of some incompatibilities with the Sphinx RTD theme (see numpy/numpydoc#215 for similar problems) your _sphinx conda environments will need rebuilt, i.e. The API page is now better populated. What do you think about inclusion? |
Closing in favour of #82. |
This is a working PR for adding documentation to Github pages. The checklist of jobs to do for completion are as follows: