Switch phase1 tracking to generic pixel CPE

[makortel]

Following the pixel offline presentation on last Friday's TRK DPG-POG meeting https://indico.cern.ch/event/536891/contributions/2376928/attachments/1374299/2086018/2016_11_18_PixelOfflineMeeting_PhaseIOfflinePlansUpdate.pdf showing a bias in the template CPE for the (final) phase1 pixel, we decided to switch the phase1 tracking (back) to generic CPE until the bias is fixed (in a way or another). This PR effectively reverts #14159.

Here are MTV plots for 1000 ttbar+35 PU events in 810pre16
https://mkortela.web.cern.ch/mkortela/tracking/validation/CMSSW_8_1_0_pre16_phase1_genericCPE
There are tiny effects in efficiencies, fake rates etc. Most visible effects are in

• resolutions
  • https://mkortela.web.cern.ch/mkortela/tracking/validation/CMSSW_8_1_0_pre16_phase1_genericCPE/ttbar_pu35_ootb/resolutionsEta.pdf
  • https://mkortela.web.cern.ch/mkortela/tracking/validation/CMSSW_8_1_0_pre16_phase1_genericCPE/ttbar_pu35_ootb/resolutionsPt.pdf
• pulls
  • https://mkortela.web.cern.ch/mkortela/tracking/validation/CMSSW_8_1_0_pre16_phase1_genericCPE/ttbar_pu35_ootb/pulls.pdf
• chi2
  • https://mkortela.web.cern.ch/mkortela/tracking/validation/CMSSW_8_1_0_pre16_phase1_genericCPE/ttbar_pu35_ootb/tuning.pdf

Tested in 8_1_0_pre16, expecting changes in 2017 workflows.

@rovere @VinInn @veszpv @boudoul

[makortel]

@cmsbuild, please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/16505/console

[cmsbuild]

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

It involves the following packages:

RecoMuon/Configuration
RecoMuon/GlobalTrackingTools
RecoMuon/MuonIdentification
RecoMuon/TrackingTools
RecoParticleFlow/PFTracking
RecoTracker/FinalTrackSelectors
RecoTracker/SpecialSeedGenerators
RecoTracker/TrackProducer

@cmsbuild, @cvuosalo, @slava77, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @felicepantaleo, @abbiendi, @dgulhan, @mmarionncern, @battibass, @jhgoh, @lgray, @HuguesBrun, @trocino, @rociovilar, @GiacomoSguazzoni, @rafaellopesdesa, @rovere, @VinInn, @bellan, @mschrode, @amagitte, @calderona, @cbernet, @gpetruc, @bachtis this is something you requested to watch as well.
@slava77, @smuzaffar you are the release manager for this.

cms-bot commands are listed here #13028

[cmsbuild]

+1
Tested at: 4d99df7
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-16703/16505/summary.html

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

Alternative comparison was/were failed for workflow(s):
25.0

[slava77]

+1

for #16703 4d99df7

• changes are as described, the same as proposed in 81X
• jenkins tests pass and comparisons show differences only in 2017 workflows

see #16740 for additional pre-signoff plots

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_9_0_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @smuzaffar

[davidlange6]

+1