Learn how easy it is to sync an existing GitHub or Google Code repo to a SourceForge project! See Demo

Close

#626 Cumulative double bond configurations (cdk-hash)

Accepted
closed
nobody
master
1
2013-05-01
2013-04-26
John May
No

Adds support to the hash code for encoding 2D and 3D cumulative double bond systems (e.g. allene and cumulene).

Discussion

    • Please add the copyright/license header for GeometricCumulativeDoubleBondFactory.java
    • can the stereo detection of these systems be extraced into a stand-alone algorithm? We don't have one yet, AFAIK
    • please add the new test class to MhashTests

    For the rest is look OK.

     
    • John May
      John May
      2013-04-29

      Doh, don't know how I forgot the license and module test inclusion.

      In theory they can be extracted, it actually just reuses the tetrahedral encoder for M/P (even number of double bonds) and the double bond encoder encoder for E/Z (for odd number). At some point I need to extract out how the parity is calculated into the cip module. It's a lot simpler and more robust but I think that's quite a big refractor/rewrite as it should also be used in SMILES and MDL. Would also probably require a few changes to the IStereoElements. I'll try and draft out what the API could be next month.

      J

       
    • Group: Needs_Review --> Needs_Revision
     
  • John May
    John May
    2013-04-30

    • Group: Needs_Revision --> Needs_Review
     
  • Thanx for the updates. Applied and pushed.

     
    • status: open --> closed
    • Group: Needs_Review --> Accepted