From: Rajarshi G. <rg...@in...> - 2007-11-27 19:19:13
|
On Nov 27, 2007, at 10:38 AM, Nina Jeliazkova wrote: > Hello all, > > It is not my pleasure to report the next bug related to Smarts, but > I was unlucky to find it first. Not at all! It's good to see what fails - makes for a stronger parser I've added 3 tests in SMARTSSearchTest (testLogicalOr[9-11]) > The NullPointerException actually is within > UniversalIsomorphismTester, which is supposed to be an old and > working code. I am not sure if LogicalOperatorBond is supposed to > have nonnull atoms, or it is > UniversalIsomorphismTester.nodeConstructor that has to verify if > atoms of a bond are non null. Any ideas? I'm not sure about this - Dazhi could verify wrt LogicalOperatorBond BTW, there's no need to create multiple instances of SMARTSQueryTool - you can reset the SMARTS for a given instance ad so reuse it ------------------------------------------------------------------- Rajarshi Guha <rg...@in...> GPG Fingerprint: 0CCA 8EE2 2EEB 25E2 AB04 06F7 1BB9 E634 9B87 56EE ------------------------------------------------------------------- Did you hear that two rabbits escaped from the zoo and so far they have only recaptured 116 of them? |