Global muon refitter and TrackTransformer

[VinInn]

as reported by Andrea Giammanco

tested with

cd CMSSW_7_1_0_pre9/src
git cms-addpkg FastSimulation/Configuration
scram b -j 8
edit FastSimulation/Configuration/python/CommonInputs_cff.py and set MixingMode = 'DigiRecoMixing' instead of 'GenMixing'

cmsDriver.py SingleMuPt10_cfi.py -s GEN,SIM,RECO --fast --pileup=mix_2012_Summer_inTimeOnly --conditions auto:startup_GRun --beamspot Realistic8TeVCollision --eventcontent=FEVTDEBUGHLT --datatier GEN-SIM-DIGI-RECO -n 10

[cmsbuild]

A new Pull Request was created by @VinInn (Vincenzo Innocente) for CMSSW_7_1_X.

Global muon refitter and TrackTransformer

It involves the following packages:

RecoMuon/GlobalTrackingTools
TrackingTools/TrackRefitter

@nclopezo, @cmsbuild, @Degano, @StoyanStoynev, @slava77 can you please review it and eventually sign? Thanks.
@GiacomoSguazzoni, @rovere, @gpetruc, @cerati, @rociovilar, @bachtis, @venturia 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.
@nclopezo, @ktf you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[giamman]

Hi all,
I just re-tested this PR, also looking at the physics correctness of a very small single-muon sample. I would say that everything is ok.

[slava77]

Hi Andrea
this is essential for 71X, right?

[giamman]

Hi Slava, I would answer yes but I am quite biased. And I don't know what the L1 management thinks about that.
Now that this PR is in, I can produce a new PR, dependent on this one, with a workflow to create a new FastSim-specific official MinBias sample needed for validation, and the workflow that will use that. In any case, coordination will be needed with PdmV in order to actually produce the new MinBias sample, etc. (My plan was to ping the PdmV conveners as soon as my PR will be on github.)
Certainly, the earlier we can start this step, the more time will be saved later. But I will let @davidlange6 comment about that.

[cmsbuild]

+1
Tested at: 610ceb9
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4179/1672/summary.html

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4179/1672/summary.html

[davidlange6]

i don't have any problem fixing bugs in 71x… hard for me to judge this small change if its in some way complex.

[giamman]

Thanks!

[slava77]

+1

for #4179 610ceb9
tested in CMSSW_7_1_X_2014-06-12-0200 (test area sign710a)

everything is OK: no diffs, timing is the same within normal jitter

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_1_X IBs unless changes (tests are also fine). @nclopezo, @ktf can you please take care of it?