Skip to content

Commit 592518f

Browse files
EcalBarrelScFiRecHits: Modify localDetFields and distance parameters (#2139)
### Briefly, what does this PR introduce? Added "sector" to CalorimeterHitReco_factory::localDetectorFields for EcalBarrelScFiRecHits so that EcalBarrelScFiRecHits will have local coordinates in sector frame of reference. With this, Island Clustering for EcalBarrelScFiProtoClusters works as expected when it is based on the localDistXZ ### What kind of change does this PR introduce? - [x ] Bug fix (issue #__) - [ ] New feature (issue #__) - [ ] Documentation update - [ ] Other: __ ### Please check if this PR fulfills the following: - [ ] Tests for the changes have been added - [ ] Documentation has been added / updated - [ ] Changes have been communicated to collaborators ### Does this PR introduce breaking changes? What changes might users need to make to their code? No ### Does this PR change default behavior? Local coordinate for ScFiRec hits will be in sector frame of reference --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
1 parent ec2793d commit 592518f

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/detectors/BEMC/BEMC.cc

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -203,7 +203,7 @@ void InitPlugin(JApplication* app) {
203203
.readout = "EcalBarrelScFiHits",
204204
.layerField = "layer",
205205
.sectorField = "sector",
206-
.localDetFields = {"system"},
206+
.localDetFields = {"system", "sector"},
207207
// here we want to use grid center position (XY) but keeps the z information from fiber-segment
208208
// TODO: a more realistic way to get z is to reconstruct it from timing
209209
.maskPos = "xy",
@@ -217,9 +217,9 @@ void InitPlugin(JApplication* app) {
217217
.adjacencyMatrix{},
218218
.peakNeighbourhoodMatrix{},
219219
.readout{},
220-
.sectorDist = 50. * dd4hep::mm,
221-
.localDistXY = {80 * dd4hep::mm, 80 * dd4hep::mm},
222-
.localDistXZ{},
220+
.sectorDist = 50. * dd4hep::mm,
221+
.localDistXY{},
222+
.localDistXZ = {80 * dd4hep::mm, 80 * dd4hep::mm},
223223
.localDistYZ{},
224224
.globalDistRPhi{},
225225
.globalDistEtaPhi{},

0 commit comments

Comments
 (0)