Skip to content

Commit 2b9c129

Browse files
committed
EventPlaneHelper: invoke geometry invariant helper once
calculateChannelCenter() was called inside GetPhiFT0 on every channel, recomputing all 208 channel positions from scratch each time despite the geometry being constant.
1 parent a97b2a1 commit 2b9c129

3 files changed

Lines changed: 2 additions & 1 deletion

File tree

Common/Core/EventPlaneHelper.cxx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,6 @@ double EventPlaneHelper::GetPhiFT0(int chno, const o2::ft0::Geometry& ft0geom)
7474
offsetY = mOffsetFT0AY;
7575
}
7676

77-
ft0geom.calculateChannelCenter();
7877
auto chPos = ft0geom.getChannelCenter(chno);
7978

8079
return TMath::ATan2(chPos.Y() + offsetY, chPos.X() + offsetX);

Common/TableProducer/qVectorsTable.cxx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -256,6 +256,7 @@ struct qVectorsTable {
256256
AxisSpec axisChID = {220, 0, 220};
257257

258258
fv0geom = o2::fv0::Geometry::instance(o2::fv0::Geometry::eUninitialized);
259+
ft0geom.calculateChannelCenter();
259260

260261
histosQA.add("ChTracks", "", {HistType::kTHnSparseF, {axisPt, axisEta, axisPhi, axixCent}});
261262
histosQA.add("FT0Amp", "", {HistType::kTH2F, {axisFITamp, axisChID}});

PWGCF/JCorran/Tasks/jEPFlowAnalysis.cxx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -444,6 +444,7 @@ struct JEPFlowAnalysis {
444444
ccdb->setCreatedNotAfter(std::chrono::duration_cast<std::chrono::milliseconds>(std::chrono::system_clock::now().time_since_epoch()).count());
445445

446446
fv0geom = o2::fv0::Geometry::instance(o2::fv0::Geometry::eUninitialized);
447+
ft0geom.calculateChannelCenter();
447448

448449
detId = getdetId(cfgDetName);
449450
refAId = getdetId(cfgRefAName);

0 commit comments

Comments
 (0)