Skip to content

use wildcards for bonds#22

Merged
rachelnwalker merged 3 commits intordkit:mainfrom
ZontaNicola:CRDGEN-378
Apr 9, 2026
Merged

use wildcards for bonds#22
rachelnwalker merged 3 commits intordkit:mainfrom
ZontaNicola:CRDGEN-378

Conversation

@ZontaNicola
Copy link
Copy Markdown
Contributor

fixed the templates so its SMILES like the others
now SMARTS will be generated with bond wildcards so they match any bond type

PLEASE DO NOT MODIFY template_smarts.h DIRECTLY

Add your templates to the templates.smi file in this repository. Add one CXSMILES per line, and please do not insert any blank lines.
Commit your changes and create the PR. The CI workflow will run on the new templates and validate them, and once they are approved, they will be automatically added to the header.

For more information please check https://github.com/rdkit/molecular_templates/blob/main/README.md

@github-actions
Copy link
Copy Markdown

#76 C12CCC(CC1)CC1CCC(CC1)C1CCC(CC3CCC(CCCCCCCCCCCC2)CC3)CC1

Comment thread template_smarts.h Outdated
@@ -18,6 +18,7 @@

// clang-format off
const std::vector<std::string> TEMPLATE_SMARTS = {
<<<<<<< Updated upstream
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this looks like it came from a diff

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

#76 C12CCC(CC1)CC1CCC(CC1)C1CCC(CC3CCC(CCCCCCCCCCCC2)CC3)CC1

@rachelnwalker
Copy link
Copy Markdown
Collaborator

@ZontaNicola there is a merge conflict, I'm pretty sure I could've just accepted current change but ill let you double check!

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 6, 2026

#76 C12CCC(CC1)CC1CCC(CC1)C1CCC(CC3CCC(CCCCCCCCCCCC2)CC3)CC1

@ZontaNicola
Copy link
Copy Markdown
Contributor Author

yes, it's the same number of lines, I'm pretty sure accepting current was the right thing to do

@rachelnwalker rachelnwalker merged commit 54aa31a into rdkit:main Apr 9, 2026
1 check passed
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