Skip to content

Commit fea6358

Browse files
authored
Merge pull request #47818 from bsunanda/Phase2-hgx363D
Phase2-hgx363D Modify the validation check of scintillation tiles to put the active cells in the geometry pool for the V19 version
2 parents bcd90ac + 86aefac commit fea6358

File tree

3 files changed

+21
-10
lines changed

3 files changed

+21
-10
lines changed

Geometry/HGCalCommonData/data/hgcalwafer/v19/hgcalwafer.xml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1197,7 +1197,7 @@
11971197
HGCalHESiliconSensitive, HGCalHEBasePlate, HGCalHEAirGap0</Vector>
11981198
<Vector name="LayerMaterials" type="string" nEntries="7">
11991199
hgcalMaterial:HGC_HEServices, hgcalMaterial:HGC_Hexaboard, materials:Epoxy,
1200-
hgcalMaterial:HGC_Kapton, materials:Silicon, hgcalMaterial:HGC_G10-FR4,
1200+
hgcalMaterial:HGC_Kapton, materials:Silicon, materials:Titanium,
12011201
materials:Air</Vector>
12021202
<Vector name="LayerThickness" type="numeric" nEntries="7">
12031203
5.060*mm, 1.300*mm, 0.125*mm, 0.300*mm, [WaferThickness200], 1.05*mm,
@@ -1235,7 +1235,7 @@
12351235
HGCalHESiliconSensitive, HGCalHEBasePlate, HGCalHEAirGap0</Vector>
12361236
<Vector name="LayerMaterials" type="string" nEntries="7">
12371237
hgcalMaterial:HGC_HEServices, hgcalMaterial:HGC_Hexaboard, materials:Epoxy,
1238-
hgcalMaterial:HGC_Kapton, materials:Silicon, hgcalMaterial:HGC_G10-FR4,
1238+
hgcalMaterial:HGC_Kapton, materials:Silicon, materials:Titanium,
12391239
materials:Air</Vector>
12401240
<Vector name="LayerThickness" type="numeric" nEntries="7">
12411241
5.060*mm, 1.300*mm, 0.125*mm, 0.300*mm, [WaferThickness200], 1.05*mm,

Geometry/HGCalCommonData/src/HGCalDDDConstants.cc

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1458,6 +1458,17 @@ int HGCalDDDConstants::tileCount(int layer, int ring) const {
14581458
bool HGCalDDDConstants::tileExist(int zside, int layer, int ring, int phi) const {
14591459
int indx = HGCalTileIndex::tileIndex(layer, ring, 0);
14601460
auto itr = hgpar_->tileInfoMap_.find(indx);
1461+
#ifdef EDM_ML_DEBUG
1462+
edm::LogVerbatim("HGCalGeomT") << "TileExist:input " << zside << ":" << layer << ":" << ring << ":" << phi
1463+
<< " Index flag " << indx << ":" << (itr != hgpar_->tileInfoMap_.end());
1464+
#endif
1465+
if (itr == hgpar_->tileInfoMap_.end()) {
1466+
indx = HGCalTileIndex::tileIndex(layer, ring, 1);
1467+
itr = hgpar_->tileInfoMap_.find(indx);
1468+
#ifdef EDM_ML_DEBUG
1469+
edm::LogVerbatim("HGCalGeomT") << "Tile Index flag " << indx << ":" << (itr != hgpar_->tileInfoMap_.end());
1470+
#endif
1471+
}
14611472
bool ok = (itr == hgpar_->tileInfoMap_.end()) ? false : HGCalTileIndex::tileExist(itr->second.hex, zside, phi);
14621473
return ok;
14631474
}
@@ -1569,8 +1580,8 @@ void HGCalDDDConstants::waferFromPosition(const double x, const double y, int& w
15691580
} else {
15701581
wafer = -1;
15711582
#ifdef EDM_ML_DEBUG
1572-
edm::LogWarning("HGCalGeom") << "Cannot get wafer type corresponding to " << x << ":" << y << " " << xx << ":"
1573-
<< yy;
1583+
edm::LogVerbatim("HGCalGeom") << "Cannot get wafer type corresponding to " << x << ":" << y << " " << xx << ":"
1584+
<< yy;
15741585
#endif
15751586
}
15761587
#ifdef EDM_ML_DEBUG

Geometry/HGCalGeometry/src/HGCalGeometry.cc

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
typedef CaloCellGeometry::Tr3D Tr3D;
2222
typedef std::vector<float> ParmVec;
2323

24-
#define EDM_ML_DEBUG
24+
//#define EDM_ML_DEBUG
2525

2626
const bool debugLocate = false;
2727

@@ -117,12 +117,12 @@ void HGCalGeometry::newCell(
117117
m_validIds.emplace_back(idc);
118118
#ifdef EDM_ML_DEBUG
119119
edm::LogVerbatim("HGCalGeom") << "Valid Id [0] " << HGCScintillatorDetId(idc);
120-
#endif
121120
} else {
122-
edm::LogWarning("HGCalGeom") << "Check " << HGCScintillatorDetId(idc) << " from " << HGCScintillatorDetId(detId)
123-
<< " Mode " << m_topology.dddConstants().geomMode() << ":" << m_topology.geomMode()
124-
<< " Valid " << m_topology.tileTrapezoid() << ":" << m_topology.valid(idc)
125-
<< " ERROR ???";
121+
edm::LogVerbatim("HGCalGeom") << "Check " << HGCScintillatorDetId(idc) << " from " << HGCScintillatorDetId(detId)
122+
<< " Mode " << m_topology.dddConstants().geomMode() << ":" << m_topology.geomMode()
123+
<< " Valid " << m_topology.tileTrapezoid() << ":" << m_topology.valid(idc)
124+
<< " ERROR ???";
125+
#endif
126126
}
127127
} else {
128128
#ifdef EDM_ML_DEBUG

0 commit comments

Comments
 (0)