-
Notifications
You must be signed in to change notification settings - Fork 92
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
Assorted documentation fixes (installation, FAQ, molecules.py, conf.py) #976
Conversation
openff/toolkit/topology/molecule.py
Outdated
@@ -5999,7 +6013,7 @@ def add_bond_charge_virtual_site(self, atoms, distance, **kwargs): | |||
|
|||
Create a bond charge-type virtual site, in which the location of the | |||
charge is specified by the position of two atoms. This supports | |||
placement of a virtual site S along a vector between two specified | |||
placement of a virtual site $S$ along a vector between two specified |
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.
This $S$
renders verbatim as written
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've spot checked a bit and this looks great. Thanks @Yoshanuikabundi. I just commented on one place where the math doesn't seem to be rendering properly, other than that it should be good to go!
We have rendering errors littered throughout our documentation. This PR is a start at going through the docs one at a time until we've got them all.
Highlights in this PR:
All the questions in the FAQ are now questions
The list of methods in the API docs now only renders once per class. Previously, it rendered once before the
__init__()
docs and repeated itself after.installation.rst
is nowinstallation.md
conf.py has had black run on it
Tag issue being addressed
Add tests
Update docstrings/documentation, if applicable
Lint codebase
Update changelog