Print

Print


Commit in lcsim/src/org/lcsim/recon/tracking/seedtracker on MAIN
MaterialManager.java+7-31.18 -> 1.19
fix isXPlane so it passes tests

lcsim/src/org/lcsim/recon/tracking/seedtracker
MaterialManager.java 1.18 -> 1.19
diff -u -r1.18 -r1.19
--- MaterialManager.java	25 May 2012 20:16:00 -0000	1.18
+++ MaterialManager.java	29 May 2012 17:32:19 -0000	1.19
@@ -44,12 +44,12 @@
  * @author Jeremy McCormick
  * @author Matt Graham
  *
- * @version $Id: MaterialManager.java,v 1.18 2012/05/25 20:16:00 mgraham Exp $
+ * @version $Id: MaterialManager.java,v 1.19 2012/05/29 17:32:19 mgraham Exp $
  */
 public class MaterialManager
 {
     // Variables from original MaterialManager class.
-    private boolean DEBUG = true; // enable debug output to System.out
+    private boolean DEBUG = false; // enable debug output to System.out
     private static final boolean TUBE_ONLY = false; // only use Tube elements
     // for calculating volume.
     private List<MaterialPolyconeSegment> _matpc = new ArrayList<MaterialPolyconeSegment>();
@@ -523,7 +523,11 @@
 
     private boolean isXPlane(double xmin, double xmax)
     {
-	return (xmax-xmin)<50.0;//5cm...this should be safe since tubes have xmin and xmax set to default huge values
+	if(xmax-xmin<0)
+	    return false; //be default xmax is negative, xmin is positive
+	if(xmax+xmin<50)
+	    return false; //catch short modules...
+	return (xmax-xmin)<50.0;//5cm...
     }
 
     public void setTransform(ITransform3D transform) {
CVSspam 0.2.12


Use REPLY-ALL to reply to list

To unsubscribe from the LCD-CVS list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=LCD-CVS&A=1