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

Reinclude functionality to enumerate unenumerated stereoisomers #3

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

althea-hansel
Copy link
Contributor

  • Added function for enumerating unenumerated stereoisomers
  • Function runs after acid/base and tautomerization, so will be performed on resulting acid/base and tautomer derivatives of initial molecule
  • Default capped at generation of 32 isomers per mol

Manual testing performed with attached Jupyter notebook
enumerate_unassigned_stereoisomers_example.ipynb.gz

@althea-hansel althea-hansel requested a review from diogomart May 8, 2024 23:16
@althea-hansel
Copy link
Contributor Author

althea-hansel commented May 28, 2024

Noticed that scrubber fails to enumerate the variants of wh9:

c1cc(ccc1CN2CCC(CC2)N3c4cc(c(cc4NC3=O)Cl)Cl)Cl

Have not found source of issue

@diogomart
Copy link
Contributor

Summary of brief meeting to tackle this issue
Here's a link to not ported and currently private code that implements such amine flips
For amines with three unique/different substituents, enumerate individual molecules, like for stereo isomers.
For amines with symmetric substituents but that are part of a ring subjected to ring flips, enumerate both amine conformers as part of ringfixes.

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.

2 participants