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

add script to convert rdkit mols to networkx graphs with attributes #32

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

Conversation

dakoner
Copy link
Contributor

@dakoner dakoner commented Nov 13, 2016

It's not clear to me this code belongs here, I'm happy to make a new repo to hold this kind of code.

@dakoner
Copy link
Contributor Author

dakoner commented Nov 13, 2016

BTW, with this script I went on to print all the 8-membered SMILES strings in GDB-13 (8.smi) along with their graph6 representation. I sorted by graph6 representation, there are some graph6 strings that are associated with only 1 8-membered SMILES string in GDB-13, for example:
CC1C2C3CC(C3)N12 GhCGjG
(it has an interesting structure)
while GhCGGC has 1406 associated SMILES strings; for example,
C=CC=CC=CC=C, which is polyacetyene(?)

Looking at 9-membered SMILES strings shows similar results.

Part of this is due to GDB-13 being a selective enumeration. So, the counts for various graphs are affected by GDB-13's aggressive filters during the graph enumeration phase.

@maxhodak
Copy link
Owner

smilesparser.py looks pretty interesting. What direction are you thinking of taking this?

@dakoner
Copy link
Contributor Author

dakoner commented Nov 25, 2016 via email

@dakoner
Copy link
Contributor Author

dakoner commented Dec 1, 2016 via email

@dakoner
Copy link
Contributor Author

dakoner commented Dec 1, 2016 via email

@rbharath
Copy link

rbharath commented Dec 1, 2016

@dakoner I'm a lurker on this repo, but very cool to see a good python SMILES parser. Do you think it would be tough to extend it to handle Reaction SMARTS? (for better support of chemical reactions)

@dakoner
Copy link
Contributor Author

dakoner commented Dec 1, 2016 via email

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.

3 participants