CSC Beam Halo filter update

[lathomas]

(Copy of the PR for 74X)
CSC Beam Halo Filter update.
-Pairs of segments in opposite endcaps are now matched with a dr/dz condition (rather than dr previously)
-New flat CSC segment matching added (the old one remains the default for now): unvetoing CSC segments associated to tracker muons with pt < 3 gev
-Optional dt matching of opposite endcaps segments
-Testing a CSC segment/calorechit matching.
-Two new working points added in BeamHaloSummary.

[cmsbuild]

A new Pull Request was created by @lathomas for CMSSW_7_5_X.

CSC Beam Halo filter update

It involves the following packages:

DataFormats/METReco
RecoMET/METAlgorithms
RecoMET/METProducers

@cmsbuild, @cvuosalo, @slava77 can you please review it and eventually sign? Thanks.
@rappoccio, @ahinzmann, @mmarionncern, @jdolen, @nhanvtran, @schoef, @mariadalfonso, @TaiSakuma 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.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[slava77]

@cmsbuild please test

@lathomas please make a PR for 76X

[cmsbuild]

The tests are being triggered in jenkins.

[cmsbuild]

Pull request #11137 was updated. @cmsbuild, @cvuosalo, @slava77 can you please check and sign again.

[cmsbuild]

+1
Tested at: 401fc58
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-11137/7899/summary.html

[cmsbuild]

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

[lathomas]

I will try to male the PR for 76X today. In the mean time, is it already possible to integrate this PR to 75X as it was done for 74X?
Thanks a lot

[cmsbuild]

Pull request #11137 was updated. @cmsbuild, @cvuosalo, @slava77 can you please check and sign again.

[cvuosalo]

@cmsbuild please test

[lathomas]

Copy of the following PR for 75X:
#11737

[cmsbuild]

Pull request #11137 was updated. @cmsbuild, @cvuosalo, @vadler, @monttj, @slava77 can you please check and sign again.

[slava77]

@cmsbuild please test

[cmsbuild]

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

[cmsbuild]

+1
Tested at: 0c4a36b
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-11137/8773/summary.html

[cmsbuild]

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

[cvuosalo]

+1

For #11137 0c4a36b

Third approval of this PR for new, small adjustments to the CSC beam halo filter that come from 76X #11737, which has already been approved by Reco. #11136 is the matching PR to this one for 74X.

The code changes are satisfactory, and Jenkins tests against baseline CMSSW_7_5_X_2015-10-12-2300 show no significant differences. An additional test of workflow 25202.0_TTbar_13 was done for #11737, and it showed no significant differences.