[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
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

Cns mpo #7915

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Cns mpo #7915

wants to merge 2 commits into from

Conversation

Adam-maz
Copy link

#CNS MPO

These scripts implement CNS-MPO calculation

…ed on the CSV file or single (multiple) input. Scripts requires SMILES and pKa values of molecule
@greglandrum
Copy link
Member

Hi @Adam-maz thanks for the contribution!

Since this relies on having a pKa value provided as part of the input, I have a hard time seeing how it is self-contained enough to be useful to the typical RDKit user.

This may be a failure of imagination on my part: what is the use case you are thinking of where a user might have SMILES for a molecule and a reliable pKa value (or prediction), but not already have the ability to calculate the CNS-MPO score?

@Adam-maz
Copy link
Author

Hello @greglandrum!! Some chemoinformatics programs (e.g. Chemdraw or Maestro Schrodinger) allow users to calculate pKa but do not offer any CNS-MPO filter. Therefore, the ready-to-use implementation of CNS-MPO in RDKit gives users the ability to easily build their own scripts to solve this problem. Additionally, I believe that implementing the CNS-MPO code is a good start for further improvement by the RDKit community.

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