Skip to content

Remove option to remove one bond or decorator #87

@bannanc

Description

@bannanc

While writing up the Reducer options for removing decorators it occurred to me that removing one or decorator at a time will never work on bonds.

For bonds, or'd together decorators in ChemPer are only the order of the bonds used to make the SMIRKS patterns. So removing one of those orders will never result in keeping clustering, where removing all of them could be ok.

For example, there are a fair number of SMIRKS in SMIRNOFF99Frosst which use double or aromatic bonds (=,:). If we need to keep a SMIRKS that matches both then trying to remove just the double decorator switching the bond to just aromatic doesn't make any sense.
However, we can imagine a situation where the order of the bond isn't actually important. In that case we could remove all of the decorators and switch the bond to and (~).

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions