From: SourceForge.net <no...@so...> - 2012-01-21 12:26:22
|
Patches item #3476966, was opened at 2012-01-21 04:26 Message generated for change (Tracker Item Submitted) made by egonw You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=3476966&group_id=20024 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: cdk-1.4.x Group: Needs Review Status: Open Resolution: None Priority: 5 Private: No Submitted By: Egon Willighagen (egonw) Assigned to: Nobody/Anonymous (nobody) Summary: Patch that sets electron count for bond orders Initial Comment: Rich made a strong point about file format limitations, and statement about decent electron counts for bonds: http://depth-first.com/articles/2012/01/11/on-the-futility-of-extending-the-molfile-format/ 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. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=3476966&group_id=20024 |