Skip to content

Commit 2b04b46

Browse files
committed
ITS: avoid range checking
Signed-off-by: Felix Schlepper <felix.schlepper@cern.ch>
1 parent 39cb913 commit 2b04b46

File tree

1 file changed

+13
-13
lines changed

1 file changed

+13
-13
lines changed

Detectors/ITSMFT/ITS/tracking/src/TrackerTraits.cxx

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -341,15 +341,15 @@ void TrackerTraits<nLayers>::computeLayerCells(const int iteration)
341341
mTimeFrame->getClusters()[iLayer][currentTracklet.firstClusterIndex].clusterId,
342342
mTimeFrame->getClusters()[iLayer + 1][nextTracklet.firstClusterIndex].clusterId,
343343
mTimeFrame->getClusters()[iLayer + 2][nextTracklet.secondClusterIndex].clusterId};
344-
const auto& cluster1_glo = mTimeFrame->getUnsortedClusters()[iLayer].at(clusId[0]);
345-
const auto& cluster2_glo = mTimeFrame->getUnsortedClusters()[iLayer + 1].at(clusId[1]);
346-
const auto& cluster3_tf = mTimeFrame->getTrackingFrameInfoOnLayer(iLayer + 2).at(clusId[2]);
344+
const auto& cluster1_glo = mTimeFrame->getUnsortedClusters()[iLayer][clusId[0]];
345+
const auto& cluster2_glo = mTimeFrame->getUnsortedClusters()[iLayer + 1][clusId[1]];
346+
const auto& cluster3_tf = mTimeFrame->getTrackingFrameInfoOnLayer(iLayer + 2)[clusId[2]];
347347
auto track{buildTrackSeed(cluster1_glo, cluster2_glo, cluster3_tf)};
348348

349349
float chi2{0.f};
350350
bool good{false};
351351
for (int iC{2}; iC--;) {
352-
const TrackingFrameInfo& trackingHit = mTimeFrame->getTrackingFrameInfoOnLayer(iLayer + iC).at(clusId[iC]);
352+
const TrackingFrameInfo& trackingHit = mTimeFrame->getTrackingFrameInfoOnLayer(iLayer + iC)[clusId[iC]];
353353

354354
if (!track.rotate(trackingHit.alphaTrackingFrame)) {
355355
break;
@@ -423,15 +423,15 @@ void TrackerTraits<nLayers>::computeLayerCells(const int iteration)
423423
mTimeFrame->getClusters()[iLayer][currentTracklet.firstClusterIndex].clusterId,
424424
mTimeFrame->getClusters()[iLayer + 1][nextTracklet.firstClusterIndex].clusterId,
425425
mTimeFrame->getClusters()[iLayer + 2][nextTracklet.secondClusterIndex].clusterId};
426-
const auto& cluster1_glo = mTimeFrame->getUnsortedClusters()[iLayer].at(clusId[0]);
427-
const auto& cluster2_glo = mTimeFrame->getUnsortedClusters()[iLayer + 1].at(clusId[1]);
428-
const auto& cluster3_tf = mTimeFrame->getTrackingFrameInfoOnLayer(iLayer + 2).at(clusId[2]);
426+
const auto& cluster1_glo = mTimeFrame->getUnsortedClusters()[iLayer][clusId[0]];
427+
const auto& cluster2_glo = mTimeFrame->getUnsortedClusters()[iLayer + 1][clusId[1]];
428+
const auto& cluster3_tf = mTimeFrame->getTrackingFrameInfoOnLayer(iLayer + 2)[clusId[2]];
429429
auto track{buildTrackSeed(cluster1_glo, cluster2_glo, cluster3_tf)};
430430

431431
float chi2{0.f};
432432
bool good{false};
433433
for (int iC{2}; iC--;) {
434-
const TrackingFrameInfo& trackingHit = mTimeFrame->getTrackingFrameInfoOnLayer(iLayer + iC).at(clusId[iC]);
434+
const TrackingFrameInfo& trackingHit = mTimeFrame->getTrackingFrameInfoOnLayer(iLayer + iC)[clusId[iC]];
435435

436436
if (!track.rotate(trackingHit.alphaTrackingFrame)) {
437437
break;
@@ -669,7 +669,7 @@ void TrackerTraits<nLayers>::processNeighbours(int iLayer, int iLevel, const bou
669669
}
670670
/// Let's start the fitting procedure
671671
CellSeed seed{currentCell};
672-
auto& trHit = mTimeFrame->getTrackingFrameInfoOnLayer(iLayer - 1).at(neighbourCell.getFirstClusterIndex());
672+
auto& trHit = mTimeFrame->getTrackingFrameInfoOnLayer(iLayer - 1)[neighbourCell.getFirstClusterIndex()];
673673

674674
if (!seed.rotate(trHit.alphaTrackingFrame)) {
675675
CA_DEBUGGER(failed[1]++);
@@ -739,7 +739,7 @@ void TrackerTraits<nLayers>::processNeighbours(int iLayer, int iLevel, const bou
739739

740740
auto seed = currentCell;
741741

742-
const auto& trHit = mTimeFrame->getTrackingFrameInfoOnLayer(iLayer - 1).at(neighbourCell.getFirstClusterIndex());
742+
const auto& trHit = mTimeFrame->getTrackingFrameInfoOnLayer(iLayer - 1)[neighbourCell.getFirstClusterIndex()];
743743
if (!seed.rotate(trHit.alphaTrackingFrame) || !propagator->propagateToX(seed, trHit.xTrackingFrame, getBz(), o2::base::PropagatorImpl<float>::MAX_SIN_PHI, o2::base::PropagatorImpl<float>::MAX_STEP, mCorrType)) {
744744
continue;
745745
}
@@ -1019,7 +1019,7 @@ void TrackerTraits<nLayers>::findShortPrimaries()
10191019
auto pvs{mTimeFrame->getPrimaryVertices(rof)};
10201020
auto pvsXAlpha{mTimeFrame->getPrimaryVerticesXAlpha(rof)};
10211021

1022-
const auto& cluster3_tf = mTimeFrame->getTrackingFrameInfoOnLayer(2).at(cluster3_glo.clusterId);
1022+
const auto& cluster3_tf = mTimeFrame->getTrackingFrameInfoOnLayer(2)[cluster3_glo.clusterId];
10231023
TrackITSExt temporaryTrack{buildTrackSeed(cluster1_glo, cluster2_glo, cluster3_tf)};
10241024
temporaryTrack.setExternalClusterIndex(0, cluster1_glo.clusterId, true);
10251025
temporaryTrack.setExternalClusterIndex(1, cluster2_glo.clusterId, true);
@@ -1085,7 +1085,7 @@ bool TrackerTraits<nLayers>::fitTrack(TrackITSExt& track, int start, int end, in
10851085
if (track.getClusterIndex(iLayer) == constants::its::UnusedIndex) {
10861086
continue;
10871087
}
1088-
const TrackingFrameInfo& trackingHit = mTimeFrame->getTrackingFrameInfoOnLayer(iLayer).at(track.getClusterIndex(iLayer));
1088+
const TrackingFrameInfo& trackingHit = mTimeFrame->getTrackingFrameInfoOnLayer(iLayer)[track.getClusterIndex(iLayer)];
10891089

10901090
if (!track.rotate(trackingHit.alphaTrackingFrame)) {
10911091
return false;
@@ -1189,7 +1189,7 @@ bool TrackerTraits<nLayers>::trackFollowing(TrackITSExt* track, int rof, bool ou
11891189
continue;
11901190
}
11911191

1192-
const TrackingFrameInfo& trackingHit = mTimeFrame->getTrackingFrameInfoOnLayer(iLayer).at(nextCluster.clusterId);
1192+
const TrackingFrameInfo& trackingHit = mTimeFrame->getTrackingFrameInfoOnLayer(iLayer)[nextCluster.clusterId];
11931193

11941194
auto tbupdated{hypo};
11951195
auto& tbuParams = outward ? tbupdated.getParamOut() : tbupdated.getParamIn();

0 commit comments

Comments
 (0)