-
Notifications
You must be signed in to change notification settings - Fork 50
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
Change log for 0.10.3 release #1236
Conversation
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.
Looks good, just a couple of minor comments. Thanks!
docs/changelog.rst
Outdated
^^^^^^^^^^^^^ | ||
|
||
* Added MPI example with dipeptide mutation by @ijpulidos in https://github.com/choderalab/perses/pull/1228 | ||
* Update comments in `RESTCapableHybridTopologyFactory` doc string by @zhang-ivy in https://github.com/choderalab/perses/pull/1189 |
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.
It is a minor thing but just to be consistent. In restructured Text the way to get inline monotyped/code blocks is with double ``
* Update comments in `RESTCapableHybridTopologyFactory` doc string by @zhang-ivy in https://github.com/choderalab/perses/pull/1189 | |
* Update comments in ``RESTCapableHybridTopologyFactory`` doc string by @zhang-ivy in https://github.com/choderalab/perses/pull/1189 |
Same applies to all the other instances.
docs/changelog.rst
Outdated
* Remove example testing by @mikemhenry in https://github.com/choderalab/perses/pull/1214 | ||
* CI miscellaneous fixes by @ijpulidos in https://github.com/choderalab/perses/pull/1217 | ||
* Improve docker building by @mikemhenry in https://github.com/choderalab/perses/pull/1200 | ||
* Clean examples -- CLI protein-ligand example for Tyk2 by @ijpulidos in https://github.com/choderalab/perses/pull/1223 |
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 think we could move this one to documentation, rather than testing.
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.
LGTM!
Cool, once it gets merged I will tag and make the release |
Description
Motivation and context
Resolves #???
How has this been tested?
Change log