Skip to content

fix: Improve handlng of heavy-atom-only inputs in RDKit conversion - #7

Open
vandan-revanur wants to merge 1 commit into
rowansci:masterfrom
vandan-revanur:fix-for-pdb-input
Open

fix: Improve handlng of heavy-atom-only inputs in RDKit conversion#7
vandan-revanur wants to merge 1 commit into
rowansci:masterfrom
vandan-revanur:fix-for-pdb-input

Conversation

@vandan-revanur

@vandan-revanur vandan-revanur commented Jul 17, 2026

Copy link
Copy Markdown

Fixes: #6

[Problem]

  • Heavy-atom-only inputs (e.g. from PDB files) were causing failures in bond assignment.
  • Previous fallback methods returned disconnected atoms, leading to incorrect SMILES representations.

[Solution]

  • Implemented a new method to create RDKit molecules from heavy-atom-only coordinates.
  • Added a retry mechanism using DetermineConnectivity when bond determination fails.

Fixes: Issue rowansci#6

[Problem]

- Heavy-atom-only inputs (e.g. from PDB files) were causing failures in bond assignment.
- Previous fallback methods returned disconnected atoms, leading to incorrect SMILES representations.

[Solution]

- Implemented a new method to create RDKit molecules from heavy-atom-only coordinates.
- Added a retry mechanism using `DetermineConnectivity` when bond determination fails.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Problem when assigning bonds from a PDB structure

1 participant