Skip to content

Commit a7246d0

Browse files
committed
GPU: Use a unified constant instead of copy and paste for 0 field cut
1 parent 919e8f2 commit a7246d0

File tree

3 files changed

+4
-3
lines changed

3 files changed

+4
-3
lines changed

GPU/Common/GPUCommonConstants.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020
namespace o2::gpu::gpu_common_constants
2121
{
2222
static constexpr const float kCLight = 0.000299792458f; // TODO: Duplicate of MathConstants, fix this now that we use only OpenCL CPP
23+
static constexpr const float kZeroFieldCut = 0.013f;
2324
}
2425

2526
#endif

GPU/GPUTracking/Merger/GPUTPCGMMerger.cxx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -378,7 +378,7 @@ void GPUTPCGMMerger::SetMaxData(const GPUTrackingInOutPointers& io)
378378
}
379379
}
380380
mNMaxOutputTrackClusters = mRec->MemoryScalers()->NTPCMergedTrackHits(mNClusters);
381-
if (CAMath::Abs(Param().polynomialField.GetNominalBz()) < (0.013f * gpu_common_constants::kCLight)) {
381+
if (CAMath::Abs(Param().polynomialField.GetNominalBz()) < (gpu_common_constants::kZeroFieldCut * gpu_common_constants::kCLight)) {
382382
mNMaxTracks = mRec->MemoryScalers()->getValue(mNTotalSectorTracks, mNTotalSectorTracks); // 0 magnetic field
383383
} else {
384384
mNMaxTracks = mRec->MemoryScalers()->NTPCMergedTracks(mNTotalSectorTracks);
@@ -1743,7 +1743,7 @@ GPUd() void GPUTPCGMMerger::CollectMergedTracks(int32_t nBlocks, int32_t nThread
17431743
p1.DzDs() = p2.DzDs();
17441744
p1.QPt() = p2.QPt();
17451745
mergedTrack.SetAlpha(p2.Alpha());
1746-
if (CAMath::Abs(Param().polynomialField.GetNominalBz()) < (0.013f * gpu_common_constants::kCLight)) {
1746+
if (CAMath::Abs(Param().polynomialField.GetNominalBz()) < (gpu_common_constants::kZeroFieldCut * gpu_common_constants::kCLight)) {
17471747
p1.QPt() = 100.f / Param().rec.bz0Pt10MeV;
17481748
}
17491749

GPU/GPUTracking/Merger/GPUTPCGMPolynomialFieldManager.cxx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -195,7 +195,7 @@ int32_t GPUTPCGMPolynomialFieldManager::GetPolynomialField(float nominalFieldkG,
195195

196196
StoredField_t type = kUnknown;
197197

198-
if (fabsf(nominalFieldkG) < 0.013f) {
198+
if (fabsf(nominalFieldkG) < gpu_common_constants::kZeroFieldCut) {
199199
type = kUniform;
200200
nominalFieldkG = 0;
201201
} else if (fabsf(fabsf(nominalFieldkG) - 5.00668f) <= fabsf(fabsf(nominalFieldkG) - 2.f)) {

0 commit comments

Comments
 (0)