lcsim/test/org/lcsim
diff -u -r1.4 -r1.5
--- Sidloi3SimplePfaTest.java 30 Nov 2010 17:52:02 -0000 1.4
+++ Sidloi3SimplePfaTest.java 11 Mar 2011 19:30:49 -0000 1.5
@@ -10,73 +10,189 @@
import org.lcsim.event.EventHeader;
import org.lcsim.event.ReconstructedParticle;
+import org.lcsim.event.Track;
+import org.lcsim.recon.ui.ReconDriverLoi;
import org.lcsim.util.Driver;
import org.lcsim.util.cache.FileCache;
import org.lcsim.util.loop.LCSimLoop;
/**
- * This test runs the PFA on a few single jet events with the sidloi3 detector.
- * It checks that the energy sum is within a reasonable range and also that the
- * sums match an answer key based on a "known good" configuration.
+ * This test runs the PFA on a few single jet events with the sidloi3 detector. It checks
+ * that the energy sum is within a reasonable range and also that the sums match an answer
+ * key based on a "known good" configuration.
*
- * @author Jeremy McCormick <[log in to unmask]>
- * @version $Id: Sidloi3SimplePfaTest.java,v 1.4 2010/11/30 17:52:02 jeremy Exp $
+ * @author jeremym
+ * @version $Id: Sidloi3SimplePfaTest.java,v 1.5 2011/03/11 19:30:49 jeremy Exp $
*/
+// TODO Add answer keys and checks for number of PFOs and Tracks found; need to figure out a reasonable tolerance.
+// TODO Figure out the total event energy from the MCParticle Tree.
+// TODO Add additional checks on energy for different particles types (e, p, gamma, K, n) and number reconstructed.
+// TODO Follow up on cause of variability in PFA answers up to 4 GeV from run to run!
public class Sidloi3SimplePfaTest extends TestCase
{
- // Answer key with values rounded to two decimal places.
- // JM Nov-29-2010
- private static final double esums[] = { 184.13, 190.13 };
-
- public void testSingleEvent() throws Exception
- {
- URL url = new URL( "http://www.lcsim.org/test/lcio/pythia_uds_nobeam_nobrem-00-500_SLIC-v2r8p3_geant4-v9r3p1_QGSP_BERT_sidloi3.slcio" );
- FileCache cache = new FileCache();
- File file = cache.getCachedFile( url );
-
- LCSimLoop loop = new LCSimLoop();
- loop.setLCIORecordSource( file );
- loop.add( new org.lcsim.recon.ui.ReconDriverLoi() );
- loop.add( new PfaCheckDriver() );
- loop.loop( 1, null );
+ // Answer key for file from URL below. --JM Feb-24-2010
+ // TODO Add all 20 events.
+ private static final double esums[] = {
+ 469.856,
+ 526.766,
+ 485.973,
+ 483.366,
+ 505.079,
+ 498.677,
+ 506.474};
+
+ // This file has 20 events. Current answer key is based on this file.
+ private static final String fileUrl = "http://www.lcsim.org/test/lcio/pythia_uds_nobeam_nobrem-0-500_SLIC-v2r9p4_geant4-v9r3p2_QGSP_BERT_sidloi3.slcio";
+
+ // Number of events to run.
+ private static final int nToRun = 7;
+
+ // Test to run the PFA and perform some simple sanity checks on the results.
+ public void testPfa() throws Exception
+ {
+ // Get the file from WWW.
+ URL url = new URL(fileUrl);
+ FileCache cache = new FileCache();
+ File file = cache.getCachedFile(url);
+
+ // Setup the loop.
+ LCSimLoop loop = new LCSimLoop();
+ loop.setLCIORecordSource(file);
+ loop.add(new TimerStartDriver());
+ loop.add(new ReconDriverLoi());
+ loop.add(new TimerStopDriver());
+ loop.add(new PfaCheckDriver(false, 500, 5.0));
+
+ // Start the run timer.
+ final long startTime = System.currentTimeMillis();
+ final long endTime;
+
+ // Run the PFA over events.
+ loop.loop(nToRun, null);
+
+ // Print the duration of run.
+ endTime = System.currentTimeMillis();
+ final long duration = endTime - startTime;
+ System.out.println("This run of " + nToRun + " events took " + duration + " millis.");
loop.dispose();
}
-
+
+ // Starts the event timer.
+ static class TimerStartDriver extends Driver
+ {
+ static long startTime;
+ public void process(EventHeader event)
+ {
+ startTime = System.currentTimeMillis();
+ }
+
+ static public long getStarttime()
+ {
+ return startTime;
+ }
+ }
+
+ // Stops the event timer and prints duration.
+ static class TimerStopDriver extends Driver
+ {
+ long endTime;
+ public void process(EventHeader event)
+ {
+ endTime = System.currentTimeMillis();
+ long duration = endTime - TimerStartDriver.getStarttime();
+ System.out.println("Event #" + event.getEventNumber() + " - Timer took " + duration + " millis.");
+ }
+ }
+
+ // Driver to perform PFA result tests.
static class PfaCheckDriver extends Driver
{
+ // Event counter.
int eventn;
-
- public void process( EventHeader event )
+
+ // Default settings.
+ boolean checkESum = false;
+ double eventEnergy = 500;
+ double keyTolerance = 5.0;
+
+ // No arg ctor that uses defaults.
+ public PfaCheckDriver()
+ {}
+
+ // ctor with arguments.
+ public PfaCheckDriver(boolean checkESum, double eventEnergy, double keyTolerance)
+ {
+ this.checkESum = checkESum;
+ this.eventEnergy = eventEnergy;
+ this.keyTolerance = keyTolerance;
+ }
+
+ // Perform PFA result checks on this event.
+ public void process(EventHeader event)
{
- List< ReconstructedParticle > particles = event.get(
- ReconstructedParticle.class,
- EventHeader.RECONSTRUCTEDPARTICLES );
- if ( particles.size() == 0 )
- throw new RuntimeException( "No ReconstructedParticles were created by the PFA." );
- double esum = 0;
- for ( ReconstructedParticle particle : particles )
+ // Get the ReconParticles.
+ List<ReconstructedParticle> particles = event.get(ReconstructedParticle.class, EventHeader.RECONSTRUCTEDPARTICLES);
+ int nparticles = particles.size();
+
+ // Make sure some ReconParticles were created.
+ assertTrue(nparticles > 0);
+
+ // Compute energy sum of RPs.
+ double reconEnergy = 0;
+ for (ReconstructedParticle particle : particles)
{
- esum += particle.getEnergy();
+ reconEnergy += particle.getEnergy();
}
- // Compute difference of total from answer key.
- double ediff = abs( esum - esums[eventn] );
-
- // Print energy sum and difference from key.
- System.out.println( "event <" + event.getEventNumber() + "> - esum <" + esum + ">, ediff <" + ediff + ">");
-
- // Check that the energy is within a 20 GeV window of the expected total.
- if ( esum < 180. || esum > 220. )
- throw new RuntimeException( "esum <" + esum + "> is more than 20 GeV off from 200 GeV event energy." );
-
- // Check that energy matches a cached value within a 3 GeV tolerance.
- // A difference here could be okay if changes are made to the PFA that
- // alter the reconstructed energy significantly. In this case, the answer
- // table should be changed to match the new, correct values.
- if ( ediff > 3.0 )
- throw new RuntimeException( "esum <" + esum + "> does not match stored value <" + esums[ eventn ] + "> within 0.01 GeV tolerance." );
+ // Make sure the ReconstructedParticles have a non-zero energy total.
+ assertTrue(reconEnergy > 0);
+ // Make sure that some Tracks were created.
+ int ntracks = event.get(Track.class, "Tracks").size();
+ assertTrue(ntracks > 0);
+
+ // Print event summary.
+ System.out.println(" nReconParticles = " + nparticles);
+ System.out.println(" nTracks = " + ntracks);
+ System.out.println(" energySum = " + reconEnergy);
+
+ // Perform energy sum checks against event energy and answer key.
+ if (checkESum)
+ {
+ // Check that event has answer key entry.
+ if (eventn <= (esums.length - 1))
+ {
+ // Print expected key value.
+ System.out.println(" energyKey = " + esums[eventn]);
+
+ // Compute difference of total E from answer key.
+ double keyDiff = reconEnergy - esums[eventn];
+
+ // Print difference from key.
+ System.out.println(" keyDiff = " + keyDiff);
+
+ // Check that energy matches the answer key within 3 GeV.
+ if (abs(keyDiff) > keyTolerance)
+ throw new RuntimeException("Energy sum " + reconEnergy + " does not match answer key value " + esums[eventn] + " within " + keyTolerance + " GeV tolerance.");
+ }
+ else
+ {
+ System.out.println(" Skipping answer key check for event " + eventn + " which has no entry!");
+ }
+
+ // Compute difference of recon energy from event energy.
+ double eventDiff = reconEnergy - eventEnergy;
+
+ // Print difference from event energy.
+ System.out.println(" eventDiff = " + eventDiff);
+
+ // Check that energy sum is within 40 GeV.
+ if (abs(eventDiff) > 40)
+ throw new RuntimeException("Energy sum " + eventDiff + " is more than 40 GeV off from event energy of " + eventEnergy + " GeV.");
+ }
+
+ // Increment event country for answer key index.
++eventn;
}
}
-}
\ No newline at end of file
+}
