Print

Print


Author: [log in to unmask]
Date: Fri Aug  5 16:17:19 2016
New Revision: 4446

Log:
mostly debug.

Modified:
    java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/detector/converter/compact/HPSTracker2017Converter.java
    java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/geometry/compact/converter/HPSTestRunTracker2014GeometryDefinition.java
    java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/geometry/compact/converter/HPSTracker2014JavaBuilder.java
    java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/geometry/compact/converter/HPSTracker2017GeometryDefinition.java
    java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/geometry/compact/converter/HPSTracker2017JavaBuilder.java
    java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/geometry/subdetector/HPSTracker2017.java
    java/branches/layer0-thin-branch/logging/src/main/resources/org/hps/logging/config/logging.properties
    java/branches/layer0-thin-branch/steering-files/src/main/resources/org/hps/steering/users/phansson/HPSTrackingDefaults.lcsim

Modified: java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/detector/converter/compact/HPSTracker2017Converter.java
 =============================================================================
--- java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/detector/converter/compact/HPSTracker2017Converter.java	(original)
+++ java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/detector/converter/compact/HPSTracker2017Converter.java	Fri Aug  5 16:17:19 2016
@@ -1,10 +1,10 @@
 package org.lcsim.detector.converter.compact;
 
 import org.jdom.Element;
-import org.lcsim.geometry.compact.converter.HPSTracker2016JavaBuilder;
+import org.lcsim.geometry.compact.converter.HPSTracker2017JavaBuilder;
 import org.lcsim.geometry.compact.converter.HPSTrackerBuilder;
 import org.lcsim.geometry.compact.converter.HPSTrackerJavaBuilder;
-import org.lcsim.geometry.subdetector.HPSTracker2016;
+import org.lcsim.geometry.subdetector.HPSTracker2017;
 
 public class HPSTracker2017Converter extends HPSTracker2014v1Converter {
 
@@ -19,7 +19,7 @@
      * initializeBuilder(org.jdom.Element)
      */
     protected HPSTrackerJavaBuilder initializeBuilder(Element node) {
-        return new HPSTracker2016JavaBuilder(_debug, node);
+        return new HPSTracker2017JavaBuilder(_debug, node);
     }
 
     /*
@@ -29,7 +29,7 @@
      * getSubdetectorType()
      */
     public Class getSubdetectorType() {
-        return HPSTracker2016.class;
+        return HPSTracker2017.class;
     }
 
     /*

Modified: java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/geometry/compact/converter/HPSTestRunTracker2014GeometryDefinition.java
 =============================================================================
--- java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/geometry/compact/converter/HPSTestRunTracker2014GeometryDefinition.java	(original)
+++ java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/geometry/compact/converter/HPSTestRunTracker2014GeometryDefinition.java	Fri Aug  5 16:17:19 2016
@@ -1343,14 +1343,22 @@
         }
     }
 
-
+    
+    
+    
+    public abstract static class BaseSensor extends HalfModuleComponent {
+    	public BaseSensor(String name, SurveyVolume mother, AlignmentCorrection alignmentCorrection, int id) {
+            super(name, mother, alignmentCorrection, id);
+    	   }
+    }
+    
     /**
      * Silicon sensor @SurveyVolume.
      * The coordinate system is located at the same position and orientation as the half-module.
      * @author Per Hansson Adrian <[log in to unmask]>
      *
      */
-    public static class Sensor extends HalfModuleComponent {
+    public static class Sensor extends BaseSensor {
         static final double length= 100.00; 
         static final double width = 40.34; 
         static final double thickness = 0.32;
@@ -1409,6 +1417,8 @@
             return length;
         }			
     }
+    
+   
 
     /**
      * Active part of the @Sensor @SurveyVolume.

Modified: java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/geometry/compact/converter/HPSTracker2014JavaBuilder.java
 =============================================================================
--- java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/geometry/compact/converter/HPSTracker2014JavaBuilder.java	(original)
+++ java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/geometry/compact/converter/HPSTracker2014JavaBuilder.java	Fri Aug  5 16:17:19 2016
@@ -7,6 +7,7 @@
 
 import org.jdom.Element;
 import org.lcsim.detector.ILogicalVolume;
+import org.lcsim.geometry.compact.converter.HPSTestRunTracker2014GeometryDefinition.BaseSensor;
 import org.lcsim.geometry.compact.converter.HPSTestRunTracker2014GeometryDefinition.Sensor;
 import org.lcsim.geometry.compact.converter.HPSTracker2014GeometryDefinition.LongHalfModuleBundle;
 import org.lcsim.geometry.compact.converter.HPSTracker2014GeometryDefinition.LongModuleBundle;
@@ -98,7 +99,8 @@
 			// Check that it had a mother
 			if(mother==null) throw new RuntimeException("Cound't find mother to module " + m.module.getName());
 
-			if(isDebug()) System.out.printf("%s: found mother %s to module %s\n", getClass().getSimpleName(),mother.getName(),m.module.getName());
+			//if(isDebug()) 
+				System.out.printf("%s: found ewfsdhf mother %s to module %s\n", getClass().getSimpleName(),mother.getName(),m.module.getName());
 			
 			// put the module in the list of objects that will be added to LCDD
 			addModule(m, mother);
@@ -108,7 +110,8 @@
 			
 		}
 		
-		if(isDebug()) System.out.printf("%s: DONE build JAVA modules\n", getClass().getSimpleName());
+		//if(isDebug()) 
+			System.out.printf("%s: DONE build JAVA modules\n", getClass().getSimpleName());
 
 		
 		//System.out.printf("%s: Built %d JAVA geometry objects\n", getClass().getSimpleName(),javaSurveyVolumes.size());
@@ -133,7 +136,7 @@
 	 * @param bundle - module to be added
 	 * @param mother - mother JAVA geometry object
 	 */
-	private void addModule(BaseModuleBundle bundle, JavaSurveyVolume mother) {
+	protected void addModule(BaseModuleBundle bundle, JavaSurveyVolume mother) {
 	    if(bundle instanceof TestRunModuleBundle) {
 	           addTestRunModule((TestRunModuleBundle) bundle, mother);
 	       } else if(bundle instanceof LongModuleBundle) {
@@ -148,7 +151,7 @@
      * @param bundle - module to be added
      * @param mother - mother LCDD geometry object
      */
-    private void addLongModule(LongModuleBundle bundle, JavaSurveyVolume mother) {
+    protected void addLongModule(LongModuleBundle bundle, JavaSurveyVolume mother) {
         // This could perhaps be fixed if there is a relation with daughters in geometry definition?
         // create the module
         JavaSurveyVolume lcddM = new JavaGhostSurveyVolume(bundle.module, mother);
@@ -162,30 +165,52 @@
 
 
 
-    private void addLongHalfModule(HalfModuleBundle bundle2, JavaSurveyVolume mother) {
-        LongHalfModuleBundle bundle = (LongHalfModuleBundle) bundle2;
+    protected void addLongHalfModule(HalfModuleBundle bundle2, JavaSurveyVolume mother) {
+        
+    	System.out.printf("what3\n");
+    	
+    	LongHalfModuleBundle bundle = (LongHalfModuleBundle) bundle2;
+    	
+    	System.out.printf("what34\n");
+    	
         // Create the half-module
         // This is not a ghost element but reflects the module 
         // concept in the old compact description
         int oldCompactModuleId = 0;
         JavaSurveyVolume lcddHM = new JavaSurveyVolume(bundle.halfModule, mother,oldCompactModuleId);
+
+        
+    	System.out.printf("what344\n");
+
         add(lcddHM);
+        
+        System.out.printf("what33\n");
+    	
         
         // ComponentNumber is taken from old geometry where it is simply a counter when adding the xml daughters to the TestRunModule.
         // It is simply 0 for sensor and 1 for carbon fiber in the old geometry 
-        int componentNumber = ((Sensor)bundle.sensor).getId();
-
+        int componentNumber = ((BaseSensor) bundle.sensor).getId();
+
+        System.out.printf("what333\n");
+    	
+        
         // create the sensor
         JavaSurveyVolume lcddS = new JavaSurveyVolume(bundle.sensor, lcddHM, componentNumber);
         add(lcddS);
 
+        
+        System.out.printf("what3333\n");
+    	
         // create the active sensor
         JavaSurveyVolume lcddAS = new JavaSurveyVolume(bundle.activeSensor, lcddS, componentNumber);
         add(lcddAS);
         
+        
+        System.out.printf("what33333\n");
+        
 //      if(isDebug()) {
-//          System.out.printf("%s: added sensor %s \n",this.getClass().getSimpleName(), lcddS.getName());
-//          System.out.printf("%s: local coordinate system\n%s\n",this.getClass().getSimpleName(), bundle.sensor.getCoord().toString());
+          System.out.printf("%s: added sensor %s \n",this.getClass().getSimpleName(), lcddS.getName());
+          System.out.printf("%s: local coordinate system\n%s\n",this.getClass().getSimpleName(), bundle.sensor.getCoord().toString());
 //          dsd
 //      }
         

Modified: java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/geometry/compact/converter/HPSTracker2017GeometryDefinition.java
 =============================================================================
--- java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/geometry/compact/converter/HPSTracker2017GeometryDefinition.java	(original)
+++ java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/geometry/compact/converter/HPSTracker2017GeometryDefinition.java	Fri Aug  5 16:17:19 2016
@@ -10,7 +10,9 @@
 import org.apache.commons.math3.geometry.euclidean.threed.Vector3D;
 import org.jdom.Element;
 import org.lcsim.geometry.compact.converter.HPSTestRunTracker2014GeometryDefinition.BaseModule;
+import org.lcsim.geometry.compact.converter.HPSTestRunTracker2014GeometryDefinition.BaseSensor;
 import org.lcsim.geometry.compact.converter.HPSTestRunTracker2014GeometryDefinition.HalfModuleComponent;
+import org.lcsim.geometry.compact.converter.HPSTestRunTracker2014GeometryDefinition.Sensor;
 
 /**
  * 
@@ -653,7 +655,7 @@
      * @author Per Hansson Adrian <[log in to unmask]>
      *
      */
-    public static class ShortSensor extends HalfModuleComponent {
+    public static class ShortSensor extends BaseSensor {
         static final double length= 10.00 + 1.00; 
         static final double width = 14.080 + 2*0.250; 
         static final double thickness = 0.250;
@@ -665,6 +667,8 @@
         public static double getSensorThickness() {
             return height;
         }
+
+        @Override
         protected void setPos() {
             
             if(debug) System.out.printf("%s: setPos for %s\n",this.getClass().getSimpleName(),getName());
@@ -680,9 +684,13 @@
             }
             
         }
+
+        @Override
         protected void setCenter() {
             setCenter(0,0,0);
         }
+
+        @Override
         protected void setBoxDim() {
             if(useSiStripsConvention) {
                 setBoxDim(width,length,thickness);
@@ -690,15 +698,23 @@
                 setBoxDim(length,thickness,width);
             }
         }
+
+        @Override
         protected double getThickness() {
             return thickness;
         }
+
+        @Override
         protected double getHeigth() {
             return thickness;
         }
+
+        @Override
         protected double getWidth() {
             return width;
         }
+
+        @Override
         protected double getLength() {
             return length;
         }			
@@ -1060,7 +1076,7 @@
      * @author Per Hansson Adrian <[log in to unmask]>
      *
      */
-    public static class ShortHalfModuleBundle extends HalfModuleBundle {
+    public static class ShortHalfModuleBundle extends LongHalfModuleBundle {
         public ShortHalfModuleBundle() {
             super();
         }

Modified: java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/geometry/compact/converter/HPSTracker2017JavaBuilder.java
 =============================================================================
--- java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/geometry/compact/converter/HPSTracker2017JavaBuilder.java	(original)
+++ java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/geometry/compact/converter/HPSTracker2017JavaBuilder.java	Fri Aug  5 16:17:19 2016
@@ -1,6 +1,10 @@
 package org.lcsim.geometry.compact.converter;
 
 import org.jdom.Element;
+import org.lcsim.geometry.compact.converter.HPSTracker2014GeometryDefinition.LongModuleBundle;
+import org.lcsim.geometry.compact.converter.HPSTracker2017GeometryDefinition.ShortModuleBundle;
+import org.lcsim.geometry.compact.converter.HPSTrackerBuilder.BaseModuleBundle;
+import org.lcsim.geometry.compact.converter.HPSTrackerGeometryDefinition.TestRunModuleBundle;
 
 public class HPSTracker2017JavaBuilder extends HPSTracker2014v1JavaBuilder {
 
@@ -10,8 +14,52 @@
 
     @Override
     public HPSTrackerGeometryDefinition createGeometryDefinition(boolean debug, Element node) {
-        return new HPSTracker2016GeometryDefinition(debug, node);
+        return new HPSTracker2017GeometryDefinition(debug, node);
     }
     
     
+
+    protected void addModule(BaseModuleBundle bundle, JavaSurveyVolume mother) {
+    	
+    	System.out.println("what");
+    	if(bundle instanceof TestRunModuleBundle) {
+    		addTestRunModule((TestRunModuleBundle) bundle, mother);
+    	} else if(bundle instanceof ShortModuleBundle) {
+    		addShortModule((ShortModuleBundle) bundle, mother);
+
+    	} else if(bundle instanceof LongModuleBundle) {
+    		addLongModule((LongModuleBundle) bundle, mother);
+    	} else {
+    		throw new RuntimeException("The bundle is of unknown class type!");
+    	}
+    	System.out.println("what2");
+    }
+
+
+
+
+
+    /**
+     * Rules for adding the LCDD module geometry.
+     * @param bundle - module to be added
+     * @param mother - mother LCDD geometry object
+     */
+    protected void addShortModule(ShortModuleBundle bundle, JavaSurveyVolume mother) {
+    	// This could perhaps be fixed if there is a relation with daughters in geometry definition?
+    	// create the module
+    	
+    	System.out.println("what22");
+    	
+    	JavaSurveyVolume lcddM = new JavaGhostSurveyVolume(bundle.module, mother);
+    	add(lcddM);
+    	if(bundle.halfModuleAxialHole!=null)  addLongHalfModule(bundle.halfModuleAxialHole,lcddM);
+    	if(bundle.halfModuleAxialSlot!=null)  addLongHalfModule(bundle.halfModuleAxialSlot,lcddM);
+    	//if(bundle.coldBlock!=null)        add(new LCDDSurveyVolume(bundle.coldBlock, lcdd, lcddM));     
+    	if(bundle.halfModuleStereoHole!=null)  addLongHalfModule(bundle.halfModuleStereoHole,lcddM);
+    	if(bundle.halfModuleStereoSlot!=null)  addLongHalfModule(bundle.halfModuleStereoSlot,lcddM);
+
+    	System.out.println("what222");
+    }
+
+
 }

Modified: java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/geometry/subdetector/HPSTracker2017.java
 =============================================================================
--- java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/geometry/subdetector/HPSTracker2017.java	(original)
+++ java/branches/layer0-thin-branch/detector-model/src/main/java/org/lcsim/geometry/subdetector/HPSTracker2017.java	Fri Aug  5 16:17:19 2016
@@ -1,15 +1,32 @@
 package org.lcsim.geometry.subdetector;
+
+import hep.graphics.heprep.HepRep;
+import hep.graphics.heprep.HepRepFactory;
 
 import org.jdom.Element;
 import org.jdom.JDOMException;
+import org.lcsim.detector.converter.heprep.DetectorElementToHepRepConverter;
 
-public class HPSTracker2017 extends HPSTracker2014v1 {
-    
-    public HPSTracker2017(Element node) throws JDOMException 
-    {
-        super(node);
+    public class HPSTracker2017 extends AbstractTracker {
+        
+        public HPSTracker2017(Element node) throws JDOMException 
+        {
+            super(node);
+        }
+
+        public void appendHepRep(HepRepFactory factory, HepRep heprep) 
+        {
+            DetectorElementToHepRepConverter.convert(getDetectorElement(), factory, heprep, -1, false, getVisAttributes().getColor());
+        }
+        
+        public boolean isEndcap() {
+            return false;
+        }
+        
+        public boolean isBarrel() {
+            return true;
+        }
+
     }
-
     
 
-}

Modified: java/branches/layer0-thin-branch/logging/src/main/resources/org/hps/logging/config/logging.properties
 =============================================================================
--- java/branches/layer0-thin-branch/logging/src/main/resources/org/hps/logging/config/logging.properties	(original)
+++ java/branches/layer0-thin-branch/logging/src/main/resources/org/hps/logging/config/logging.properties	Fri Aug  5 16:17:19 2016
@@ -18,15 +18,15 @@
 java.util.logging.ConsoleHandler.formatter = java.util.logging.SimpleFormatter
 
 # lcsim job
-org.lcsim.job.level = CONFIG
+org.lcsim.job.level = FINEST
 org.lcsim.job.EventMarkerDriver.level = ALL
 org.lcsim.job.EventPrintLoopAdapter = ALL
 
 # conditions
-org.hps.conditions.api.level = WARNING
+org.hps.conditions.api.level = FINEST
 org.hps.conditions.database.level = FINEST
-org.hps.conditions.cli.level = CONFIG
-org.hps.conditions.ecal.level = WARNING
+org.hps.conditions.cli.level = FINEST
+org.hps.conditions.ecal.level = FINEST
 org.hps.conditions.svt.level = FINEST
 
 # monitoring-drivers

Modified: java/branches/layer0-thin-branch/steering-files/src/main/resources/org/hps/steering/users/phansson/HPSTrackingDefaults.lcsim
 =============================================================================
--- java/branches/layer0-thin-branch/steering-files/src/main/resources/org/hps/steering/users/phansson/HPSTrackingDefaults.lcsim	(original)
+++ java/branches/layer0-thin-branch/steering-files/src/main/resources/org/hps/steering/users/phansson/HPSTrackingDefaults.lcsim	Fri Aug  5 16:17:19 2016
@@ -21,7 +21,6 @@
     <driver name="ConditionsDriver" type="org.hps.conditions.ConditionsDriver">
          <detectorName>${detector}</detectorName>
           <runNumber>10000</runNumber>
-          <!--<runNumber>${run}</runNumber>-->
         <freeze>true</freeze>
     </driver>
         <driver name="EventMarkerDriver" type="org.lcsim.job.EventMarkerDriver">