Strip cluster shape filter with subcluster info

Introduce a new strip cluster shape filter:
  * in the trajectory building of the initial, pixelPair and detachedTriplet steps, only for the TIB and TID layers 1,2
  * in the seed building for pixelLess (replacing the old one) and tobTec (where there was none)

Presented at the tracking POG meeting on 22 Sep 2014, https://indico.cern.ch/event/337840/
After the meeting, it was verified that it doesn't have ill effects on the QCD 3TeV relval, and that the small inefficiency introduced in the pixelLess and tobTec does not depend on the transverse impact parameter of the tracks.
I can do all these small changes, but i'd wait for the tests to be complete
(they won't have a dramatic impact on timing)

     Giovanni
Il 26-set-2014 15:24 "Vincenzo Innocente" <notifications@github.com> ha
scritto:

> In
> RecoPixelVertexing/PixelLowPtUtilities/src/StripSubClusterShapeTrajectoryFilter.cc:
>
> > +        StripSubClusterShapeFilterBase(iConfig,iC),
> > +        filterAtHelixStage_(iConfig.getParameter<bool>("FilterAtHelixStage"))
> > +
> > +{
> > +    if (filterAtHelixStage_) edm::LogError("Configuration") << "StripSubClusterShapeSeedFilter: FilterAtHelixStage is not yet working correctly.\n";
> > +}
> > +
> > +
> > +/*****************************************************************************/
> > +bool StripSubClusterShapeSeedFilter::compatible
> > +  (const  TrajectoryStateOnSurface & tsos, SeedingHitSet::ConstRecHitPointer thit) const
> > +{
> > +   if (filterAtHelixStage_) return true;
> > +   const TrackingRecHit* hit = thit->hit();
> > +   if (hit == 0 || !hit->isValid()) return true;
> > +   if (hit->geographicalId().subdetId() <= 2) return true; // we look only at strips for now
>
> 3 is called SiStripDetId::TIB (2 is FPX ?)
> so
> hit->geographicalId().subdetId() < SiStripDetId::TIB
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cms-sw/cmssw/pull/5583/files#r18087583>.
>