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

RDKit reaction produces wrong double bond stereochemistry #4410

Closed
mwojcikowski opened this issue Aug 11, 2021 · 3 comments · Fixed by #4414
Closed

RDKit reaction produces wrong double bond stereochemistry #4410

mwojcikowski opened this issue Aug 11, 2021 · 3 comments · Fixed by #4414
Labels
Milestone

Comments

@mwojcikowski
Copy link
Contributor

For a quite basic reaction RDKit since 2020.09 produces wrong stereochemistry on double bond. Last know version that worked correctly is 2020.03.

In following reaction latest RDKit (2021.03) is producing chlorines in cis position instead of trans.
image

Code to reproduce

from rdkit.Chem import AllChem
from rdkit import Chem
rxn = AllChem.ReactionFromSmarts('[N:4][C:6](/[Cl:9])=[C:7](\\[Cl:10])[C:11].[Br:5]>>[N:4].[Br:5][CX3:6](\\[Cl:9])=[CX3:7](/[Cl:10])[C:11]')
res = rxn.RunReactants((Chem.MolFromSmiles('C\\C(Cl)=C(\\N)Cl'), Chem.MolFromSmiles('[Br]')))
assert Chem.MolToSmiles(res[0][1]) == 'C/C(Cl)=C(\\Cl)Br'

Version info:

  • RDKit version: 2021.03
  • OS: MacOS Big Sur
  • Python version (if relevant): 3.8
  • Are you using conda? Yes
  • If you are using conda, which channel did you install the rdkit from? conda-forge

Pinging @ricrogz

@ricrogz
Copy link
Contributor

ricrogz commented Aug 11, 2021

I think the reaction is correct: in your product, [Br:5] has a higher CIP priority than [Cl:9], and since it is on the same side of the double bond as [Cl:10], the bond is labeled as Z, as Z/E nomenclature always uses the highest CIP neighbors as reference.

@mwojcikowski
Copy link
Contributor Author

@ricrogz I didn't mean absolute configuration, rather relative position of chlorine atoms.
Maybe a drawing will be more precise.
Produced:
image

Expected:
image

@ricrogz
Copy link
Contributor

ricrogz commented Aug 11, 2021

Sorry, you are right, something is wrong. I had just had a quick look at your initial message in my email, and somehow got the impression that the image depicted the problem, not the correct reaction. I'll look into it.

@ricrogz ricrogz mentioned this issue Aug 11, 2021
greglandrum pushed a commit that referenced this issue Aug 13, 2021
* consider reversed bonds

* add test

* check smiles
@greglandrum greglandrum added this to the 2021_03_5 milestone Aug 13, 2021
greglandrum pushed a commit that referenced this issue Aug 24, 2021
* consider reversed bonds

* add test

* check smiles
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants