Bug fix to avoid duplicated clusters on the same track in upgraded FPIX

This pull request includes a bug fix and a few changes in an analysis package
I think it affects only upgrade geometry, despite the file has the same name in SLHC and standard releases. In the near future the names will be different to make clear that they are about different geometries (and to allow different geometries in the same release).


On Oct 20, 2014, at 8:46 , Vincenzo Innocente <notifications@github.com<mailto:notifications@github.com>>
 wrote:


In RecoTracker/TkDetLayers/src/PixelForwardLayer.cc:

> @@ -203,7 +203,7 @@ PixelForwardLayer::groupedCompatibleDetsV( const TrajectoryStateOnSurface& tsos,
>    if(!closestResult_outer.empty()){
>      DetGroupElement closestGel( closestResult_outer.front().front());
>      float window = computeWindowSize( closestGel.det(), closestGel.trajectoryState(), est);
> -    searchNeighbors( tsos, prop, est, crossings_inner, window, result_outer, false);
> +    searchNeighbors( tsos, prop, est, crossings_outer, window, result_outer, false);


is this a long standing bug? should be propagated to 7_2_X and 7_3_X?
do we know the impact on Run1?

—
Reply to this email directly or view it on GitHub<https://github.com/cms-sw/cmssw/pull/5889/files#r19069736>.