bugfix for pixel clustershape filter, and, change HI tracking low-pt seeding from triplet to track

As listed in the commits, first commit is for bugfixing the clustershape filter, removing conflict between two vs three argument version of the operator(). This was a prerequisite for the update of low-pt tracking.

In the second commit, the HI tracking uses tracks instead of triplets in seeding, which is a significant improvement in timing. Details in: https://indico.cern.ch/event/345510/contribution/5/material/slides/0.pdf

This PR requires https://github.com/cms-sw/cmssw/pull/6374 to be merged, in order to run the full HI reco sequence successfully.


No, this is just tells you whether the class inherits from 
clusterShapeTrackFilter or not.
If it does, it should use the 3 parameter version of the operator(), 
whether or not the parameter is set to True.
There is certainly a more elegant solution, but this was the minimally 
invasive approach.



On 11/13/14 3:52 PM, Matti Kortelainen wrote:
>
> In RecoPixelVertexing/PixelTrackFitting/src/PixelTrackReconstruction.cc:
>
> > @@ -56,6 +56,7 @@ PixelTrackReconstruction::PixelTrackReconstruction(const ParameterSet& cfg,
> >      if(theConfig.exists("useFilterWithES")) {
> >        edm::LogInfo("Obsolete") << "useFilterWithES parameter is obsolete and can be removed";
> >      }
> > +    useClusterShape = filterPSet.exists("useClusterShape");
>
> Should also the value of useClusterShape parameter be checked (in 
> addition of its existence)?
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/cms-sw/cmssw/pull/6376/files#r20295348>.
>


-- 
-----------------------
Matthew Nguyen
Chargé de Recherche
LLR-Ecole Polytechnique
91128 Palaiseau FRANCE
+33 1 69 33 55 65
-----------------------