-
Notifications
You must be signed in to change notification settings - Fork 438
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…s conventions to a single place
Changes Unknown when pulling 303480f on murrayrm:doc-update into ** on python-control:master**. |
Changes Unknown when pulling 93b50b1 on murrayrm:doc-update into ** on python-control:master**. |
Changes Unknown when pulling 349025b on murrayrm:doc-update into ** on python-control:master**. |
From commit 1f15678, 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. |
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 to master
branch.
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 to my earlier comment where I describe 1f15678 as the only such commit. |
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.