<?xml version="1.0" encoding="UTF-8"?>
<commit>
  <added type="array">
    <added>
      <filename>data/pubchem_416_benzenes.sdf</filename>
    </added>
  </added>
  <modified type="array">
    <modified>
      <diff>@@ -4,21 +4,21 @@
   &lt;param name=&quot;japex.classPath&quot; value=&quot;build/classes&quot;/&gt;
   &lt;param name=&quot;japex.resultUnit&quot; value=&quot;ms&quot;/&gt;
   &lt;param name=&quot;japex.warmupIterations&quot; value=&quot;10&quot;/&gt;
-  &lt;param name=&quot;japex.runIterations&quot; value=&quot;10&quot;/&gt;
+  &lt;param name=&quot;japex.runIterations&quot; value=&quot;5&quot;/&gt;
   &lt;param name=&quot;japex.reportsDirectory&quot; value=&quot;build/reports&quot;/&gt;
   &lt;driverGroup name=&quot;Ring Benchmark&quot; xmlns=&quot;http://www.sun.com/japex/testSuite&quot;&gt;
     &lt;driver name=&quot;CDK-1.2.x&quot;&gt;
       &lt;param name=&quot;japex.driverClass&quot; value=&quot;net.openmolecules.benchmark.driver.CDKRingBench&quot;/&gt;
       &lt;param name=&quot;japex.classPath&quot; value=&quot;${basedir}/lib/cdk-1.2.x-20090116.jar&quot;/&gt;
-      &lt;param name=&quot;description&quot; value=&quot;Reads SD file, counts all rings in each structure.&quot;/&gt;
+      &lt;param name=&quot;description&quot; value=&quot;Reads SD file, finds all rings in each hydrogen-suppressed structure.&quot;/&gt;
     &lt;/driver&gt;
     &lt;driver name=&quot;mx-0.108.1&quot;&gt;
       &lt;param name=&quot;japex.driverClass&quot; value=&quot;net.openmolecules.benchmark.driver.MXRingBench&quot;/&gt;
       &lt;param name=&quot;japex.classPath&quot; value=&quot;${basedir}/lib/mx-0.108.1.jar&quot;/&gt;
-      &lt;param name=&quot;description&quot; value=&quot;Reads SD file, counts all rings in each structure.&quot;/&gt;
+      &lt;param name=&quot;description&quot; value=&quot;Reads SD file, finds all rings in each hydrogen-suppressed structure.&quot;/&gt;
     &lt;/driver&gt;
   &lt;/driverGroup&gt;
-  &lt;testCase name=&quot;CDK and MX Read 33-Record SD File and Count Rings in Each Structure&quot;&gt;
-    &lt;param name=&quot;japex.inputFile&quot; value=&quot;data/steroids.sdf&quot; /&gt;
+  &lt;testCase name=&quot;Read 416-Record PubChem SD File and Find All Rings in Each Hydrogen-Suppressed Structure&quot;&gt;
+    &lt;param name=&quot;japex.inputFile&quot; value=&quot;data/pubchem_416_benzenes.sdf&quot; /&gt;
   &lt;/testCase&gt;
 &lt;/testSuite&gt;
\ No newline at end of file</diff>
      <filename>mx-cdk-ring.xml</filename>
    </modified>
    <modified>
      <diff>@@ -21,7 +21,6 @@
  * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
  * THE SOFTWARE.
  */
-
 package net.openmolecules.benchmark.driver;
 
 import com.sun.japex.JapexDriverBase;
@@ -45,19 +44,17 @@ import org.openscience.cdk.tools.manipulator.AtomContainerManipulator;
 public class CDKRingBench extends JapexDriverBase
 {
   private List&lt;IAtomContainer&gt; molecules;
-  private AllRingsFinder finder;
 
   @Override
   public void prepare(TestCase testCase)
   {
     molecules = new ArrayList();
-    finder = new AllRingsFinder();
     IteratingMDLReader reader = getReader(testCase.getParam(&quot;japex.inputFile&quot;));
 
     while (reader.hasNext())
     {
       IAtomContainer molecule = (IMolecule) reader.next();
-      
+
       molecule = AtomContainerManipulator.removeHydrogens(molecule);
       molecules.add(molecule);
     }
@@ -66,7 +63,6 @@ public class CDKRingBench extends JapexDriverBase
     {
       reader.close();
     }
-    
     catch (IOException e)
     {
       throw new RuntimeException(e);
@@ -76,20 +72,19 @@ public class CDKRingBench extends JapexDriverBase
   @Override
   public void run(TestCase testCase)
   {
-    int sum = 0;
+    AllRingsFinder finder = new AllRingsFinder();
 
     for (IAtomContainer molecule : molecules)
     {
       try
       {
-        sum += finder.findAllRings(molecule).getAtomContainerCount();
-      } catch (CDKException e)
+        finder.findAllRings(molecule);
+      }
+      catch (CDKException e)
       {
         throw new RuntimeException(e);
       }
     }
-    
-    System.out.println(&quot;sum=&quot; + sum);
   }
 
   private IteratingMDLReader getReader(String filename)
@@ -101,7 +96,8 @@ public class CDKRingBench extends JapexDriverBase
       Reader raw = new FileReader(filename);
 
       result = new IteratingMDLReader(raw, DefaultChemObjectBuilder.getInstance());
-    } catch (Exception e)
+    }
+    catch (Exception e)
     {
       throw new RuntimeException(e);
     }</diff>
      <filename>src/net/openmolecules/benchmark/driver/CDKRingBench.java</filename>
    </modified>
    <modified>
      <diff>@@ -41,14 +41,12 @@ public class MXRingBench extends JapexDriverBase
 {
 
   private List&lt;Molecule&gt; molecules;
-  private RingFinder finder;
 
   @Override
   public void prepare(TestCase testCase)
   {
     molecules = new ArrayList();
     SDFileReader reader = createReader(testCase.getParam(&quot;japex.inputFile&quot;));
-    finder = new HanserRingFinder();
 
     while (reader.hasNextRecord())
     {
@@ -65,14 +63,12 @@ public class MXRingBench extends JapexDriverBase
   @Override
   public void run(TestCase testCase)
   {
-    int sum = 0;
+    RingFinder finder = new HanserRingFinder();
     
     for (Molecule molecule : molecules)
     {
-      sum += finder.findRings(molecule).size();
+      finder.findRings(molecule);
     }
-    
-    System.out.println(&quot;sum=&quot; + sum);
   }
 
   private SDFileReader createReader(String filename)</diff>
      <filename>src/net/openmolecules/benchmark/driver/MXRingBench.java</filename>
    </modified>
  </modified>
  <removed type="array"/>
  <parents type="array">
    <parent>
      <id>a6b59dea87d931878393f2024444f296d18b127b</id>
    </parent>
  </parents>
  <author>
    <name>Rich Apodaca</name>
    <email>rapodaca@metamolecular.com</email>
  </author>
  <url>http://github.com/rapodaca/cheminfbenchmark/commit/b4390dd9c4b69d331cf7bc49dd96d9e1ac415428</url>
  <id>b4390dd9c4b69d331cf7bc49dd96d9e1ac415428</id>
  <committed-date>2009-01-21T08:57:45-08:00</committed-date>
  <authored-date>2009-01-21T08:57:45-08:00</authored-date>
  <message>changed ring test data to be more representative</message>
  <tree>331d1544ed4bfc4dc111f5534745eeb9901d6aa0</tree>
  <committer>
    <name>Rich Apodaca</name>
    <email>rapodaca@metamolecular.com</email>
  </committer>
</commit>
