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.

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

Assign Molecule's connectivity with an RDKit template #51

Closed
martimunicoy opened this issue Sep 8, 2020 · 1 comment · Fixed by #52 or #50
Closed

Assign Molecule's connectivity with an RDKit template #51

martimunicoy opened this issue Sep 8, 2020 · 1 comment · Fixed by #52 or #50
Labels
enhancement New feature or request
Projects

Comments

@martimunicoy
Copy link
Owner

Description

If we want to load PDB files with no connectivity data (such as those coming from PELE simulations) we need a method to assign the missing connectivity.

Solution

We can use an RDKit molecule representation with the correct connectivity assigned as the connectivity template. Then, the RDKit's method AllChem.AssignBondOrdersFromTemplate can be employed to assign the connectivity to a third molecular representation using the previous connectivity template.

@martimunicoy martimunicoy added the enhancement New feature or request label Sep 8, 2020
@martimunicoy martimunicoy added this to To do in v0.3.1 via automation Sep 8, 2020
This was linked to pull requests Sep 8, 2020
@j-wags
Copy link

j-wags commented Sep 14, 2020

I just recently saw that MDAnalysis has added a to_rdkit method, and that they put some solid work into guessing valid connectivities + bond orders. It may be worth checking this out:

MDAnalysis PR: MDAnalysis/mdanalysis#2916
Blog post from author about guessing connectivity: https://cedric.bouysset.net/blog/2020/07/22/rdkit-converter-part2

v0.3.1 automation moved this from To do to Done Sep 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
No open projects
v0.3.1
  
Done
Development

Successfully merging a pull request may close this issue.

2 participants