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 to_smiles function #6038

Merged
merged 5 commits into from
Nov 23, 2022
Merged

Add to_smiles function #6038

merged 5 commits into from
Nov 23, 2022

Conversation

hichiaty
Copy link
Contributor

This PR:

  • Adds a PyG Graph to_smiles function based on the from_smiles function.
  • Adds further options to x_map and e_map to match RDkit's map indices
  • Fixes kekulize in the from_smiles function
  • Adds test to check from_smiles and to_smiles are consistent

Copy link
Member

@rusty1s rusty1s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@rusty1s rusty1s changed the title Add to_smiles function to utils/smiles.py and tests Add to_smiles function Nov 23, 2022
@rusty1s rusty1s enabled auto-merge (squash) November 23, 2022 15:34
@rusty1s rusty1s merged commit f543b6a into pyg-team:master Nov 23, 2022
JakubPietrakIntel pushed a commit to JakubPietrakIntel/pytorch_geometric that referenced this pull request Nov 25, 2022
This PR:

- Adds a PyG Graph `to_smiles` function based on the `from_smiles`
function.
- Adds further options to `x_map` and `e_map` to match RDkit's map
indices
- Fixes kekulize in the `from_smiles` function
- Adds test to check `from_smiles` and `to_smiles` are consistent

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: rusty1s <matthias.fey@tu-dortmund.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants