Skip to content

Commit b8bbbc6

Browse files
committed
Merge remote-tracking branch 'origin/development' into dev-align3
2 parents 86fbc53 + 2e0fcc1 commit b8bbbc6

File tree

3 files changed

+18
-8
lines changed

3 files changed

+18
-8
lines changed

common-tools/clas-jcsg/src/main/java/org/jlab/detector/geom/RICH/RICHGeoCalibration.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ public void load_CCDB(ConstantsManager manager, int run, int ncalls, RICHGeoPara
8989

9090
if((debugMode>=1 || geopar.DEBUG_GEO_CONSTS>=1) && ncalls<Math.max(1,geopar.DEBUG_GEO_CONSTS)) {
9191
System.out.format("------------------------------------------------------------- \n");
92-
System.out.format("RICH: Load ALI Calibration from local TxT file for RICH 4d sector %4d run %6d \n", irich, isec, run);
92+
System.out.format("RICH: Load ALI Calibration from local TxT file for RICH %4d sector %4d run %6d \n", irich, isec, run);
9393
System.out.format("------------------------------------------------------------- \n");
9494

9595
dump_AliCalibration(isec, "TXT ");
@@ -102,7 +102,7 @@ public void load_CCDB(ConstantsManager manager, int run, int ncalls, RICHGeoPara
102102

103103
if((debugMode>=1 || geopar.DEBUG_GEO_CONSTS>=1) && ncalls<=Math.max(1,geopar.DEBUG_GEO_CONSTS)) {
104104
System.out.format("------------------------------------------------------------- \n");
105-
System.out.format("RICH: Load AER Calibration from local TxT file for RICH 4d sector %4d run %6d \n", irich, isec, run);
105+
System.out.format("RICH: Load AER Calibration from local TxT file for RICH %4d sector %4d run %6d \n", irich, isec, run);
106106
System.out.format("------------------------------------------------------------- \n");
107107

108108
dump_AerCalibration(isec, "TXT ");

common-tools/clas-jcsg/src/main/java/org/jlab/detector/geom/RICH/RICHGeoFactory.java

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,6 @@ public class RICHGeoFactory{
3838

3939
private RICHGeant4Factory richfactory = new RICHGeant4Factory();
4040
private RICHPixelMap pixelmap = new RICHPixelMap();
41-
private RICHPixel pmtpixels = null;
4241
private RICHGeoParameters geopar = new RICHGeoParameters();
4342
private RICHGeoCalibration geocal = new RICHGeoCalibration();
4443

@@ -87,7 +86,7 @@ public RICHGeoFactory(int FactoryMode, ConstantsManager manager, int run, boolea
8786
geocal.load_CCDB(manager, run, Ncalls, geopar);
8887

8988
if(FactoryMode>0){
90-
// global pixel coordinate indexes
89+
// global pixel coordinate indexes (obsolete)
9190
pixelmap.init_GlobalPixelGeo();
9291

9392
// RICH survey (obsolete)
@@ -577,15 +576,16 @@ public int select_AeroforTraj(int isec, Line3D first, Line3D second, Line3D thir
577576

578577

579578
//------------------------------
580-
public Vector3d GetPixelCenter(int ipmt, int anode){
579+
/*public Vector3d GetPixelCenter(int ipmt, int anode){
581580
//------------------------------
582581
582+
// obsolete as refers to un-aligned richfactory
583583
Vector3d Vertex = richfactory.GetPhotocatode(ipmt).getVertex(2);
584584
Vector3d VPixel = Vertex.plus(pmtpixels.GetPixelCenter(anode));
585585
//System.out.format("Std vtx %8.3f %8.3f %8.3f \n",Vertex.x, Vertex.y, Vertex.z);
586586
return new Vector3d (VPixel.x, -VPixel.y, VPixel.z);
587587
588-
}
588+
}*/
589589

590590

591591
//------------------------------
@@ -598,7 +598,7 @@ public Point3D get_Pixel_Center(int isec, int ipmt, int anode){
598598
Face3D compo_face = get_Layer(isec, ilay).get_CompoFace(ipmt-1, 0);
599599
Vector3d Vertex = toVector3d( compo_face.point(1) );
600600

601-
Vector3d VPixel = Vertex.plus(pmtpixels.GetPixelCenter(anode));
601+
Vector3d VPixel = Vertex.plus(get_Layer(isec, ilay).get_PMTPixels().GetPixelCenter(anode));
602602
return new Point3D (VPixel.x, -VPixel.y, VPixel.z);
603603
}
604604

@@ -1347,7 +1347,8 @@ public void generate_PixelMap(RICHLayer layer, int ico, Shape3D compo_plane, Lis
13471347
}
13481348

13491349
if(downversor!=null && rightversor!= null) {
1350-
pmtpixels = new RICHPixel(new Vector3d(0.,0.,0.), downversor, rightversor);
1350+
RICHPixel pmtpixels = new RICHPixel(new Vector3d(0.,0.,0.), downversor, rightversor);
1351+
layer.set_PMTPixels(pmtpixels);
13511352
if(debugMode>=1){
13521353
pmtpixels.show_Pixels( vertex );
13531354
vertex = toVector3d( layer.get_CompoFace(5,0).point(1) );

common-tools/clas-jcsg/src/main/java/org/jlab/detector/geom/RICH/RICHLayer.java

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@ public class RICHLayer extends ArrayList<RICHComponent> {
4242
private ArrayList<Integer> compo_list = new ArrayList<Integer>();
4343

4444
private RICHFrame local_frame = new RICHFrame();
45+
private RICHPixel pmtpixels = null;
4546

4647

4748
// ----------------
@@ -99,6 +100,14 @@ public RICHLayer(int isec, int ilay, RICHGeoParameters geopar) {
99100
public int get_size() { return this.size(); }
100101
// ----------------
101102

103+
// ----------------
104+
public void set_PMTPixels(RICHPixel pmtpixels){ this.pmtpixels = pmtpixels;}
105+
// ----------------
106+
107+
// ----------------
108+
public RICHPixel get_PMTPixels(){ return this.pmtpixels;}
109+
// ----------------
110+
102111
// ----------------
103112
public Shape3D get_TrackingSurf() {
104113
// ----------------

0 commit comments

Comments
 (0)