From 78b5481fd0ce4aa9daa769c3bb30a21f3f603686 Mon Sep 17 00:00:00 2001 From: Sebouh Paul Date: Wed, 7 Aug 2024 19:08:54 -0400 Subject: [PATCH] lowered the amount of distance neighbor hits in topoclustering in the ZDC (#1532) ### Briefly, what does this PR introduce? Redefines neighboring subcell hits in the ZDC (in the context of topo clustering) to be the nearest neighbors only for hits in the same layer, and the same position for hits in adjacent layers. This change reduces the cases where showers from different particles are merged into a single cluster ### What kind of change does this PR introduce? - [ ] Bug fix (issue #__) - [ ] New feature (issue #__) - [ ] Documentation update - [X] Other: parameter tweak ### Please check if this PR fulfills the following: - [X] Tests for the changes have been added - [X] Documentation has been added / updated - [X] 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? yes. Changes the default behavior of the topo clustering in the ZDC --------- Co-authored-by: Dmitry Kalinkin Co-authored-by: Derek M Anderson --- src/detectors/ZDC/ZDC.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/detectors/ZDC/ZDC.cc b/src/detectors/ZDC/ZDC.cc index 3a3ae384e..22c034ffe 100644 --- a/src/detectors/ZDC/ZDC.cc +++ b/src/detectors/ZDC/ZDC.cc @@ -158,8 +158,8 @@ extern "C" { "HcalFarForwardZDCImagingProtoClusters", {"HcalFarForwardZDCSubcellHits"}, {"HcalFarForwardZDCImagingProtoClusters"}, { .neighbourLayersRange = 1, - .localDistXY = {0.76*side_length, 0.76*side_length*sin(M_PI/3)}, - .layerDistXY = {0.76*side_length, 0.76*side_length*sin(M_PI/3)}, + .localDistXY = {0.5*side_length, 0.5*side_length*sin(M_PI/3)}, + .layerDistXY = {0.25*side_length, 0.25*side_length*sin(M_PI/3)}, .layerMode=eicrecon::ImagingTopoClusterConfig::ELayerMode::xy, .sectorDist = 10.0 * dd4hep::cm, .minClusterHitEdep = 100.0 * dd4hep::keV,