#471 Patch that sets electron count for bond orders

master (162)

Rich made a strong point about file format limitations, and statement about decent electron counts for bonds:


Since the CDK supports this, I guess it was about time to actually start using it...

So, this patch automatically sets electron count when a particular bond order is chosen.

I prefer this in cdk-1.4.x as it should have set this from the start, and while the electron count = 0 right now, I doubt any code is in fact using it.

Still, being an API change, I prefer two reviewers.


  • Rajarshi Guha

    Rajarshi Guha - 2012-01-21

    the first 2 patches look fine; but patch 3 seems to be corrupted as nothing gets downloaded

  • Egon Willighagen

    Downloading the 3rd patch works fine for me, but I had yesterday also trouble downloading a patch from SF, which only could be downloaded half.. I guess the same problem.

    But before we can continue this patch, we first need to resolve that debug logger thingy...

  • Egon Willighagen

    Rajarshi, I have uploaded a new 0002 and 0003 that no longer depends on the datadebug patch, by introducing a private helper method to set the electron counts based on the bond order.

  • Egon Willighagen

    Rajarshi, can you please check the updates?

    Also, let's just put this in master, not cdk-1.4.x...

  • Rajarshi Guha

    Rajarshi Guha - 2012-05-18

    OK, looks good now. Applied and pushed. Put it into master


Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:

No, thanks