Skip to content

Commit 6449540

Browse files
authored
[PWGJE] Fixing jetCorrelationD0 table columns (#15365)
1 parent bca6d9b commit 6449540

File tree

1 file changed

+11
-13
lines changed

1 file changed

+11
-13
lines changed

PWGJE/Tasks/jetCorrelationD0.cxx

Lines changed: 11 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -27,10 +27,6 @@
2727
#include "Framework/HistogramRegistry.h"
2828
#include "Framework/Logger.h"
2929
#include "Framework/runDataProcessing.h"
30-
#include <Framework/ASoA.h>
31-
#include <Framework/HistogramSpec.h>
32-
33-
#include <fairlogger/Logger.h>
3430

3531
#include <string>
3632
#include <vector>
@@ -57,8 +53,10 @@ DECLARE_SOA_TABLE(McCollisionTables, "AOD", "MCCOLLINFOTABLE",
5753

5854
namespace collisionInfo
5955
{
60-
DECLARE_SOA_INDEX_COLUMN(CollisionTable, collisionTable);
61-
DECLARE_SOA_INDEX_COLUMN(McCollisionTable, mcCollisionTable);
56+
// DECLARE_SOA_INDEX_COLUMN(CollisionTable, collisionTable);
57+
DECLARE_SOA_INDEX_COLUMN_CUSTOM(CollisionTable, collisionTable, "COLLINFOTABLES");
58+
// DECLARE_SOA_INDEX_COLUMN(McCollisionTable, mcCollisionTable);
59+
DECLARE_SOA_INDEX_COLUMN_CUSTOM(McCollisionTable, mcCollisionTable, "MCCOLLINFOTABLES");
6260
} // namespace collisionInfo
6361
namespace d0Info
6462
{
@@ -91,7 +89,7 @@ DECLARE_SOA_TABLE(D0DataTables, "AOD", "D0DATATABLE",
9189
d0Info::D0Phi,
9290
d0Info::D0Y);
9391

94-
DECLARE_SOA_TABLE(D0McPTables, "AOD", "D0MCPARTICLELEVELTABLE",
92+
DECLARE_SOA_TABLE(D0McPTables, "AOD", "D0MCPTABLE",
9593
o2::soa::Index<>,
9694
collisionInfo::McCollisionTableId,
9795
d0Info::D0McOrigin,
@@ -103,8 +101,8 @@ DECLARE_SOA_TABLE(D0McPTables, "AOD", "D0MCPARTICLELEVELTABLE",
103101
namespace jetInfo
104102
{
105103
// D0 tables
106-
DECLARE_SOA_INDEX_COLUMN(D0DataTable, d0Data);
107-
DECLARE_SOA_INDEX_COLUMN(D0McPTable, d0MCP);
104+
DECLARE_SOA_INDEX_COLUMN(D0DataTable, d0DataTable);
105+
DECLARE_SOA_INDEX_COLUMN(D0McPTable, d0McPTable);
108106
// Jet
109107
DECLARE_SOA_COLUMN(JetPt, jetPt, float);
110108
DECLARE_SOA_COLUMN(JetEta, jetEta, float);
@@ -202,7 +200,7 @@ struct JetCorrelationD0 {
202200
{
203201
for (const auto& jetBase : jetsBase) {
204202
if (jetBase.has_matchedJetGeo()) { // geometric matching
205-
for (auto& jetTag : jetBase.template matchedJetGeo_as<std::decay_t<U>>()) {
203+
for (auto const& jetTag : jetBase.template matchedJetGeo_as<std::decay_t<U>>()) {
206204
registry.fill(HIST("hPtMatched"), jetBase.pt() - (rho * jetBase.area()), jetTag.pt(), weight);
207205
registry.fill(HIST("hPtMatched1d"), jetTag.pt(), weight);
208206
registry.fill(HIST("hPhiMatched"), jetBase.phi(), jetTag.phi(), weight);
@@ -280,7 +278,7 @@ struct JetCorrelationD0 {
280278
continue;
281279
}
282280
float dphi = RecoDecay::constrainAngle(jet.phi() - d0Candidate.phi());
283-
if (abs(dphi - M_PI) > (M_PI / 2)) { // this is quite loose instead of pi/2 could do 0.6
281+
if (std::abs(dphi - o2::constants::math::PI) > (o2::constants::math::PI / 2)) { // this is quite loose instead of pi/2 could do 0.6
284282
continue;
285283
}
286284
fillJetHistograms(jet, dphi);
@@ -323,7 +321,7 @@ struct JetCorrelationD0 {
323321
continue;
324322
}
325323
float dphi = RecoDecay::constrainAngle(jet.phi() - d0Candidate.phi());
326-
if (abs(dphi - M_PI) > (M_PI / 2)) { // this is quite loose instead of pi/2 could do 0.6
324+
if (std::abs(dphi - o2::constants::math::PI) > (o2::constants::math::PI / 2)) { // this is quite loose instead of pi/2 could do 0.6
327325
continue;
328326
}
329327
fillJetHistograms(jet, dphi);
@@ -362,7 +360,7 @@ struct JetCorrelationD0 {
362360
continue;
363361
}
364362
float dphi = RecoDecay::constrainAngle(jet.phi() - d0MCPCandidate.phi());
365-
if (abs(dphi - M_PI) > (M_PI / 2)) {
363+
if (std::abs(dphi - o2::constants::math::PI) > (o2::constants::math::PI / 2)) {
366364
continue;
367365
}
368366
fillJetHistograms(jet, dphi);

0 commit comments

Comments
 (0)