Fix PixelTrackCleanerWrapper for quadruplet SeedingHitSet

[makortel]

PixelTrackCleanerWrapper assumed SeeingHitSets have 2 or 3 hits, resulting in pixel tracks that have 3 hits although they were reconstructed from quadruplets. Fixed by storing also the 4th hit if there is one.

Should not affect on pt/eta/etc of pixel tracks because the cleaning is done after the fit, but distributions sensitive to the number of hits may be affected.

FYI @thokc @cerati @rovere @VinInn

[cmsbuild]

A new Pull Request was created by @makortel (Matti Kortelainen) for CMSSW_6_2_X_SLHC.

Fix PixelTrackCleanerWrapper for quadruplet SeedingHitSet

It involves the following packages:

RecoPixelVertexing/PixelTrackFitting

@nclopezo, @cmsbuild, @Degano, @StoyanStoynev, @slava77 can you please review it and eventually sign? Thanks.
@cerati, @GiacomoSguazzoni, @rovere this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
@fratnikov, @mark-grimes you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[VinInn]

ping...
no test?

[ktf]

I just started it.

On 26 Jun 2014, at 14:48, Vincenzo Innocente wrote:

ping...
no test?

---

Reply to this email directly or view it on GitHub:
#4372 (comment)

[cmsbuild]

+1
Tested at: 5e3fd84
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4372/3/summary.html

[fratnikov]

merge
tested on single pions. Known problems in harvesting for HGCal configurations