Skip to content

to_rdkit_mol should have no_implicit option #42

@richardzhu

Description

@richardzhu

If we are maintaining a tinygraph obj that keeps track of hydrogens using the explicit_h prop, then if we export to RDKit mol using to_rdkit_mol we should also also set no_implicit on the atoms that have it, right? I was thinking something like

if explicit_h_prop is not None:
    atom.SetNumExplicitHs(int(g.v[explicit_h_prop][i]))
    atom.SetNoImplicit(True)

if explicit_h_prop is not None:

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions