Skip to content

Commit 65e32db

Browse files
committed
modified SCS and minor changes in the SuperclusteringDNNInputs
1 parent cb9937e commit 65e32db

File tree

2 files changed

+12
-8
lines changed

2 files changed

+12
-8
lines changed

RecoHGCal/TICL/interface/SuperclusteringDNNInputs.h

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
// Author: Theo Cuisset - [email protected]
33
// Date: 11/2023
44

5-
// Modified by Gamze Sokmen
5+
// Modified by Gamze Sokmen - [email protected]
66
// Changes: Implementation of the delta time feature under a new DNN input version (v3) for the superclustering DNN and correcting the seed pT calculation.
77
// Date: 07/2025
88

@@ -14,6 +14,11 @@
1414
#include <memory>
1515

1616
namespace ticl {
17+
18+
// any raw_dt outside +/- kDeltaTimeDefault is considered bad
19+
static constexpr float kDeltaTimeDefault = 50.f;
20+
static constexpr float kBadDeltaTime = -5.f;
21+
1722
class Trackster;
1823

1924
// Abstract base class for DNN input preparation.
@@ -91,7 +96,7 @@ namespace ticl {
9196
}
9297
};
9398

94-
/* Third version of DNN by Gamze Sokmen, making use of time information as new variables.
99+
/* Third version of DNN by Gamze Sokmen and Shamik Ghosh, making use of time information as new variables.
95100
Uses features : ['DeltaEta', 'DeltaPhi', 'multi_en', 'multi_eta', 'multi_pt', 'seedEta','seedPhi','seedEn', 'seedPt', theta', 'theta_xz_seedFrame', 'theta_yz_seedFrame', 'theta_xy_cmsFrame', 'theta_yz_cmsFrame', 'theta_xz_cmsFrame', 'explVar', 'explVarRatio', 'mod_deltaTime']
96101
*/
97102

@@ -126,4 +131,4 @@ namespace ticl {
126131
std::unique_ptr<AbstractSuperclusteringDNNInput> makeSuperclusteringDNNInputFromString(std::string dnnVersion);
127132
} // namespace ticl
128133

129-
#endif
134+
#endif

RecoHGCal/TICL/src/SuperclusteringDNNInputs.cc

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
// Author: Theo Cuisset - [email protected]
33
// Date: 11/2023
44

5-
// Modified by Gamze Sokmen
5+
// Modified by Gamze Sokmen - [email protected]
66
// Changes: Implementation of the delta time feature under a new DNN input version (v3) for the superclustering DNN and correcting the seed pT calculation.
77
// Date: 07/2025
88

@@ -123,8 +123,8 @@ namespace ticl {
123123

124124
float explVar_denominator = std::accumulate(
125125
std::begin(ts_toCluster.eigenvalues()), std::end(ts_toCluster.eigenvalues()), 0.f, std::plus<float>());
126-
float explVarRatio = 0.;
127-
if (explVar_denominator != 0.) {
126+
float explVarRatio = 0.f;
127+
if (explVar_denominator != 0.f) {
128128
explVarRatio = ts_toCluster.eigenvalues()[0] / explVar_denominator;
129129
} else {
130130
edm::LogWarning("HGCalTICLSuperclustering")
@@ -134,8 +134,7 @@ namespace ticl {
134134

135135
// modified deltaTime: set the default values <-50 or >50 to -5
136136
float raw_dt = ts_toCluster.time() - ts_base.time();
137-
float mod_deltaTime = (raw_dt < -50.f || raw_dt > 50.f) ? -5.f : raw_dt;
138-
137+
float mod_deltaTime = ( raw_dt < -kDeltaTimeDefault || raw_dt > kDeltaTimeDefault ) ? kBadDeltaTime : raw_dt;
139138

140139
return {{
141140
std::abs(ts_toCluster.barycenter().Eta()) - std::abs(ts_base.barycenter().Eta()), // DeltaEtaBaryc

0 commit comments

Comments
 (0)