openff-toolkit icon indicating copy to clipboard operation
openff-toolkit copied to clipboard

Have `Molecule.canonical_order_atoms` optionally return a mapping from the old to the new atom order

Open j-wags opened this issue 4 years ago • 1 comments

This would unblock an important feature in fragmenter. Full description at https://github.com/openforcefield/openff-fragmenter/issues/129

I think the best way to do this is to thread a new kwarg through Molecule, RDKitToolkitWrapper, and OpenEyeToolkitWrapper's canonical_order_atoms methods.

j-wags avatar Nov 06 '21 01:11 j-wags

+1 to this, and also, ensure that map indices survive the trip through this method. See also here for an example implementation.

SimonBoothroyd avatar Nov 06 '21 08:11 SimonBoothroyd