Reduce fake rate loose tracks

[cerati]

Bug fixes for these two problems resulting in higher fakerate for generalTracks collection (when highPurity selection is not made):

1. Now we have highPurity without corresponding loose/tight selection => need to explicitly set loose and tight when highPurity is passed
2. New highPurity selection had flag keepAllTracks=True so that, for some iterations, tracks not even passing loose cuts were saved in generalTracks collection

For more info:
https://indico.cern.ch/getFile.py/access?contribId=23&sessionId=5&resId=0&materialId=slides&confId=295861

This is meant to go in 700 (and 71X of course)

[cmsbuild]

A new Pull Request was created by @cerati for CMSSW_7_0_X.

Reduce fake rate loose tracks

It involves the following packages:

RecoTracker/FinalTrackSelectors
RecoTracker/IterativeTracking

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

[cmsbuild]

+1
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-2151/2153/summary.html

[slava77]

Here are some notes from comparisons in CMSSW_7_0_X_2014-01-20-1400 (using 05ce46e of this PR)
part-1: fwlite scripts (not DQM)

[ @cerati @rovere Giuseppe and Marco, please comment on the muon iteration migration noted below]

In general changes are as expected:

• higher pileup samples have larger drop in general tracks, even more visible in the conversion step tracks
• pflow and egamma objects don't change much (thanks to preselection)

Unclear or somewhat suspicious changes:

• for hadron or electron-dominated samples the generalTracks are suppressed primarily in the algo=9 (iter5).
  e.g. in TTBar with 2012-like PU (wflow 202)
  
• muon gun samples have a drop in algo=10 as well (OK), but there is a major migration of algo=4 (iter0) tracks to algo=13, the muon-seeded iteration, e.g., muon gun pt=20 GeV
  
  A visible side-effect for this is that the stored MVA value is now predominantly not computed (-99).
• TeV muon sample now has a significant increase in electron seeds with fairly long tracks (is it due to some preference to iterations used for ele seeding?) The number of particle flow EGamma candidate electrons actually goes down (more efficient rejection by overlap)
  
• high-pu sample (TTBar 2012-PU here) show a large decrease in PF displaced vertices at .. still consistent with reduction of fakes
  

[slava77]

small follow-up to the above:
DQM plot of the track quality (qualityMask() ) in mu pt=10 GeV (correction to the above: change pt=20 to pt=10 geV). This looks like "confirmed" bit is not set in this case.
Should this be happening?

[slava77]

+1

tested eb11f25 in CMSSW_7_0_X_2014-01-20-1400 (sign299a test area).
The last commit fixed the odd distribution mentioned above.

This PR is for 700pre13

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_0_X IBs unless changes or unless it breaks tests. @ktf can you please take care of it?

[cmsbuild]

+1
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-2151/11/summary.html

[cmsbuild]

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