Skip to content

Commit

Permalink
RecoTracker: Changes in GroupedCkfTrajectoryBuilder from 6_2_0_SLHC21
Browse files Browse the repository at this point in the history
Code commented with annotation
"fixme gc patched for SLHC - skip this check to avoid overlaps (to be tested for standard geom)"
  • Loading branch information
makortel committed Dec 5, 2014
1 parent 107dd98 commit 16cd901
Showing 1 changed file with 8 additions and 6 deletions.
14 changes: 8 additions & 6 deletions RecoTracker/CkfPattern/plugins/GroupedCkfTrajectoryBuilder.cc
Original file line number Diff line number Diff line change
Expand Up @@ -856,11 +856,12 @@ GroupedCkfTrajectoryBuilder::rebuildSeedingRegion(const TrajectorySeed&seed,
// (e.g. because no Tracker layers outside seeding region)
//

if ( it->measurements().size()<=startingTraj.measurements().size() ) {
rebuiltTrajectories.push_back(std::move(*it));
LogDebug("CkfPattern")<< "RebuildSeedingRegion skipped as in-out trajectory does not exceed seed size.";
continue;
}
//fixme gc patched for SLHC - skip this check to avoid overlaps (to be tested for standard geom)
//if ( it->measurements().size()<=startingTraj.measurements().size() ) {
//rebuiltTrajectories.push_back(std::move(*it));
//LogDebug("CkfPattern")<< "RebuildSeedingRegion skipped as in-out trajectory does not exceed seed size.";
//continue;
//}
//
// Refit - keep existing trajectory in case fit is not possible
// or fails
Expand Down Expand Up @@ -1021,7 +1022,8 @@ GroupedCkfTrajectoryBuilder::backwardFit (TempTrajectory& candidate, unsigned in
// skip candidates which are not exceeding the seed size
// (e.g. Because no Tracker layers exist outside seeding region)
//
if unlikely( candidate.measurements().size()<=nSeed ) return TempTrajectory();
//fixme gc patched for SLHC - skip this check to avoid overlaps (to be tested for standard geom)
//if unlikely( candidate.measurements().size()<=nSeed ) return TempTrajectory();

LogDebug("CkfPattern")<<"nSeed " << nSeed << endl
<< "Old traj direction = " << candidate.direction() << endl
Expand Down

1 comment on commit 16cd901

@makortel
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes coming from cms-sw#3051 (ad2d37b), both the PR and commit touch also RecoTracker/TkDetLayers/src/PixelBlade.cc (see review commit 1397a8d).

Since marked fixme, maybe a proper solution would be needed before including to 74X?

Please sign in to comment.