From: SourceForge.net <no...@so...> - 2010-09-26 08:08:21
|
Patches item #3074735, was opened at 2010-09-24 13:03 Message generated for change (Comment added) made by egonw You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=3074735&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: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Egon Willighagen (egonw) Assigned to: Nobody/Anonymous (nobody) Summary: Implementation of Acidic and Basic group count descriptors Initial Comment: For feature request #3056330. ---------------------------------------------------------------------- >Comment By: Egon Willighagen (egonw) Date: 2010-09-26 10:08 Message: Rajarshi, thanx for your review. The SMARTS are derived from JOELib, per the feature request, but will add that information. I also left a question on BOx [0]. Unfortunately, the JOELib JavaDoc is not more informative about the source of those SMARTS. I assume those two ring system are corner cases, and will fix the JavaDoc. Regarding the larger molecule, I can add a e.g. this: http://pubchem.ncbi.nlm.nih.gov/summary/summary.cgi?cid=151489&loc=ec_rcs Is that what you have a in mind? A larger, curated test set would of course be ideal, but I am not aware of such. 0.http://blueobelisk.shapado.com/questions/paper-with-smarts-based-acidic-and-basic-group-counts ---------------------------------------------------------------------- Comment By: Rajarshi Guha (rajarshi) Date: 2010-09-26 00:14 Message: Please include sources (if any) for the SMARTS patterns In the acidic groups descriptor it's not clear what $(n1nnnc1) and $(n1nncn1) are supposed to match. Could you include an example in the Javadocs (or bring the Javadocs in sync with the code as these are commented out) A few more comprehensive (ie bigger target SMILES) tests would be useful otherwise looks good ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=3074735&group_id=20024 |