- Notifications
You must be signed in to change notification settings - Fork446
Small documentation updates#123
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Conversation
…s conventions to a single place
coveralls commentedDec 27, 2016 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Changes Unknown when pulling303480f on murrayrm:doc-update into ** on python-control:master**. |
coveralls commentedDec 27, 2016 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Changes Unknown when pulling93b50b1 on murrayrm:doc-update into ** on python-control:master**. |
coveralls commentedDec 29, 2016 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Changes Unknown when pulling349025b on murrayrm:doc-update into ** on python-control:master**. |
slivingston commentedDec 31, 2016
From commit1f15678, names from I think that the change is reasonable because there have not been collisions among desired names of function or classes (at least, not that I am aware of). Furthermore, the change is consistent with the current pattern of organization. |
murrayrm commentedDec 31, 2016
The change in |
| `lapack` library. More information on the slycot package can be obtained | ||
| from the `slycot project page<https://github.com/python-control/Slycot>`_. | ||
| For users with a working the Anaconda distribution of Python, the following |
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.
Should this be "...with the Anaconda distribution..."?
@murrayrm If you agree, I can merge and then delete the words "a working".
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.
The change is trivial, if the text should indeed be different, and can be applied directly tomaster branch.
slivingston commentedDec 31, 2016
I just noticed another commit involving more than documentation:93b50b1.Your comment above applies here as well. OK to include the change. I mention it here only as correction tomy earlier comment where I describe1f15678 as the only such commit. |
slivingston commentedDec 31, 2016
There are several instances of trailing whitespace, but they are small and only in documentation, so I decided to merge and avoid further delay. |
This PR includes some small updates to the python-control documentation, including updating the path for Slycot, adding some entries for missing functions, and adding a new "conventions" section.