Skip to content

Commit 2ca3ec9

Browse files
committed
GPU TPC: Remove some obsolete code and track members (leftover from Run 2 by Sergey and totally forgotten), which were wasting performance
1 parent 4b72f18 commit 2ca3ec9

File tree

2 files changed

+1
-29
lines changed

2 files changed

+1
-29
lines changed

GPU/GPUTracking/Merger/GPUTPCGMMergedTrack.h

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,6 @@ class GPUTPCGMMergedTrack
4141
{
4242
return mAlpha;
4343
}
44-
GPUd() float LastX() const { return mLastX; }
45-
GPUd() float LastY() const { return mLastY; }
46-
GPUd() float LastZ() const { return mLastZ; }
4744
GPUd() bool OK() const { return mFlags & 0x01; }
4845
GPUd() bool Looper() const { return mFlags & 0x02; }
4946
GPUd() bool CSide() const { return mFlags & 0x04; }
@@ -55,9 +52,6 @@ class GPUTPCGMMergedTrack
5552
GPUd() void SetFirstClusterRef(int32_t v) { mFirstClusterRef = v; }
5653
GPUd() void SetParam(const GPUTPCGMTrackParam& v) { mParam = v; }
5754
GPUd() void SetAlpha(float v) { mAlpha = v; }
58-
GPUd() void SetLastX(float v) { mLastX = v; }
59-
GPUd() void SetLastY(float v) { mLastY = v; }
60-
GPUd() void SetLastZ(float v) { mLastZ = v; }
6155
GPUd() void SetOK(bool v)
6256
{
6357
if (v) {
@@ -110,9 +104,6 @@ class GPUTPCGMMergedTrack
110104
gputpcgmmergertypes::GPUTPCOuterParam mOuterParam; //* outer param
111105

112106
float mAlpha; //* alpha angle
113-
float mLastX; //* outer X
114-
float mLastY; //* outer Y
115-
float mLastZ; //* outer Z
116107
uint32_t mFirstClusterRef; //* index of the first track cluster in corresponding cluster arrays
117108
// TODO: Change to 8 bit
118109
uint32_t mNClusters; //* number of track clusters

GPU/GPUTracking/Merger/GPUTPCGMTrackParam.cxx

Lines changed: 1 addition & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -1143,26 +1143,7 @@ GPUd() void GPUTPCGMTrackParam::RefitTrack(GPUTPCGMMergedTrack& GPUrestrict() tr
11431143
track.Param() = t;
11441144
track.Alpha() = Alpha;
11451145

1146-
if (track.OK()) {
1147-
int32_t ind = track.FirstClusterRef();
1148-
const GPUParam& GPUrestrict() param = merger->Param();
1149-
float alphaa = param.Alpha(merger->Clusters()[ind].sector);
1150-
float xx, yy, zz;
1151-
if (merger->Param().par.earlyTpcTransform) {
1152-
xx = merger->ClustersXYZ()[ind].x;
1153-
yy = merger->ClustersXYZ()[ind].y;
1154-
zz = merger->ClustersXYZ()[ind].z - track.Param().GetTZOffset();
1155-
} else {
1156-
const ClusterNative& GPUrestrict() cl = merger->GetConstantMem()->ioPtrs.clustersNative->clustersLinear[merger->Clusters()[ind].num];
1157-
merger->GetConstantMem()->calibObjects.fastTransformHelper->Transform(merger->Clusters()[ind].sector, merger->Clusters()[ind].row, cl.getPad(), cl.getTime(), xx, yy, zz, track.Param().GetTZOffset());
1158-
}
1159-
float sinA, cosA;
1160-
CAMath::SinCos(alphaa - track.Alpha(), sinA, cosA);
1161-
track.SetLastX(xx * cosA - yy * sinA);
1162-
track.SetLastY(xx * sinA + yy * cosA);
1163-
track.SetLastZ(zz);
1164-
// merger->DebugRefitMergedTrack(track);
1165-
}
1146+
// if (track.OK()) merger->DebugRefitMergedTrack(track);
11661147
}
11671148

11681149
GPUd() void GPUTPCGMTrackParam::Rotate(float alpha)

0 commit comments

Comments
 (0)