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

Poor result for constrained coord generation in rapamycin derivative. #110

Open
DavidACosgrove opened this issue Feb 8, 2022 · 1 comment

Comments

@DavidACosgrove
Copy link

Hi,
I am calling CoordGen via RDKit, and have had a problem with constrained generation of coords for a rapamycin analogue using rapamycin as the template. The discussion is here: rdkit/rdkit#4994 (reply in thread). It was suggested that I file a bug report here. I hope the cross-link is a suitable way of doing it. I used the SMARTS string [#6]1(-[#6]2(-[#6](-[#6]-[#6]-[#6](-[#8]-2)-[#6]-[#6](-[#6](-[#6])=[#6]-[#6]=[#6]-[#6]=[#6]-[#6](-[#6])-[#6]-[#6](-[#6](-[#6](-[#6](-[#6](=[#6]-[#6](-[#6](-[#6]-[#6](-[#8]-[#6](-[#6]2-[#7](-[#6]-1=[#8])-[#6]-[#6]-[#6]-[#6]-2)=[#8])-[#6](-[#6])-[#6]-[#6]1-[#6]-[#6]-[#6](-[#6](-[#6]-1)-[#8]-[#6])-[#8])=[#8])-[#6])-[#6])-[#8])-[#8]-[#6])=[#8])-[#6])-[#8]-[#6])-[#6])-[#8])=[#8] as the reference pattern, which is that RDKit returns for the MCS for the two structures.
It's worth saying, I think, that the CoordGen algorithm lays out macrocycles like this way better than any other toolkit I've seen.

@d-b-w
Copy link
Collaborator

d-b-w commented Feb 9, 2022

Thanks for the report, @DavidACosgrove.

There may be something that we should double check in RDKit's invocation of coordgen, but this is most likely an issue strictly on the coordgen side.

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

No branches or pull requests

2 participants