Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Remove confId from *FFOptimizeMoleculeConfs Python docs #2805

Closed
ptosco opened this issue Nov 19, 2019 · 0 comments
Closed

Remove confId from *FFOptimizeMoleculeConfs Python docs #2805

ptosco opened this issue Nov 19, 2019 · 0 comments
Assignees
@ptosco
Labels
bug
Milestone
2019_09_3

Comments

@ptosco
Copy link
Contributor

ptosco commented Nov 19, 2019

The documentation for UFFOptimizeMoleculeConfs and MMFFOptimizeMoleculeConfs mentions a confId parameter which does not exist in the C++ backend and needs to be removed.
@ptosco ptosco added the bug label Nov 19, 2019
@ptosco ptosco self-assigned this Nov 19, 2019
ptosco added a commit to ptosco/rdkit that referenced this issue Nov 19, 2019
@ptosco
Fixes rdkit#2805
a53a321
@ptosco ptosco mentioned this issue Nov 19, 2019
Merged
@greglandrum greglandrum modified the milestones: 2019_09_2, 2019_09_3 Nov 29, 2019
greglandrum pushed a commit that referenced this issue Jan 9, 2020
@ptosco @greglandrum
Fixes #2805 (#2806)
9551fa3 
* Fixes #2805

* - removed confId unused parameters from helpers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ptosco ptosco

Labels
bug
Projects
None yet
Milestone
2019_09_3
Development

No branches or pull requests
2 participants
@greglandrum @ptosco