New Medium muon ID selectors and offline DQM plots

[trocino]

This is to include a new "Medium" muon selector for reco::Muon and pat::Muon. The new ID selection was proposed and studied by @gpetruc and @cbotta [1], and is optimized to have efficiency close to Loose muons and fake rate at the same level as Tight muons.
For monitoring purposes, few plots were included in muon offline DQM. A couple of plots are attached here, comparing eta and p distributions of Loose, Medium, and Tight muons.

[1] https://indico.cern.ch/event/357213/contribution/2/material/slides/0.pdf

[cmsbuild]

A new Pull Request was created by @trocino (Daniele Trocino) for CMSSW_7_5_X.

New Medium muon ID selectors and offline DQM plots

It involves the following packages:

DQMOffline/Muon
DataFormats/MuonReco
DataFormats/PatCandidates

@nclopezo, @cvuosalo, @monttj, @cmsbuild, @deguio, @slava77, @vadler, @danduggan can you please review it and eventually sign? Thanks.
@barvic, @battibass, @abbiendi, @jhgoh, @bellan, @amagitte, @bachtis, @rociovilar 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.
@nclopezo 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

it's very nice to see both the code change in reco/selectors as well as a DQM plot for that at the same time.

[cmsbuild]

The tests are being triggered in jenkins.

[cmsbuild]

-1
Tested at: 6d2b431
I found a compilation error while trying to compile with clang:
I used this command:
scram b vclean && scram build -k -j 8 USER_CXXFLAGS='-fsyntax-only' COMPILER='llvm compile'

>> Compiling  /build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_7_5_X_2015-04-24-1100/src/ElectroWeakAnalysis/ZMuMu/bin/zChi2Fit.cpp 
warning: unknown warning option '-Wno-psabi' [-Wunknown-warning-option]
In file included from /build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_7_5_X_2015-04-24-1100/src/ElectroWeakAnalysis/ZMuMu/bin/zFitToyMc.cpp:11:
In file included from /afs/cern.ch/cms/sw/ReleaseCandidates/vol1/slc6_amd64_gcc491/cms/cmssw-patch/CMSSW_7_5_X_2015-04-24-1100/src/PhysicsTools/Utilities/interface/RootMinuit.h:12:
In file included from /afs/cern.ch/cms/sw/ReleaseCandidates/vol1/slc6_amd64_gcc491/lcg/root/6.02.00-odfocd2/include/Math/SMatrix.h:88:
/afs/cern.ch/cms/sw/ReleaseCandidates/vol1/slc6_amd64_gcc491/lcg/root/6.02.00-odfocd2/include/Math/MatrixRepresentationsStatic.h:162:24: fatal error: recursive template instantiation exceeded maximum depth of 256
      typedef typename make_indices_impl,
                       ^
/afs/cern.ch/cms/sw/ReleaseCandidates/vol1/slc6_amd64_gcc491/lcg/root/6.02.00-odfocd2/include/Math/MatrixRepresentationsStatic.h:162:24: note: in instantiation of template class 'ROOT::Math::rowOffsetsUtils::make_indices_impl<251, ROOT::Math::rowOffsetsUtils::indices<0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26, 27, 28, 29, 30, 31, 32, 33, 34, 35, 36, 37, 38, 39, 40, 41, 42, 43, 44, 45, 46, 47, 48, 49, 50, 51, 52, 53, 54, 55, 56, 57, 58, 59, 60, 61, 62, 63, 64, 65, 66, 67, 68, 69, 70, 71, 72, 73, 74, 75, 76, 77, 78, 79, 80, 81, 82, 83, 84, 85, 86, 87, 88, 89, 90, 91, 92, 93, 94, 95, 96, 97, 98, 99, 100, 101, 102, 103, 104, 105, 106, 107, 108, 109, 110, 111, 112, 113, 114, 115, 116, 117, 118, 119, 120, 121, 122, 123, 124, 125, 126, 127, 128, 129, 130, 131, 132, 133, 134, 135, 136, 137, 138, 139, 140, 141, 142, 143, 144, 145, 146, 147, 148, 149, 150, 151, 152, 153, 154, 155, 156, 157, 158, 159, 160, 161, 162, 163, 164, 165, 166, 167, 168, 169, 170, 171, 172, 173, 174, 175, 176, 177, 178, 179, 180, 181, 182, 183, 184, 185, 186, 187, 188, 189, 190, 191, 192, 193, 194, 195, 196, 197, 198, 199, 200, 201, 202, 203, 204, 205, 206, 207, 208, 209, 210, 211, 212, 213, 214, 215, 216, 217, 218, 219, 220, 221, 222, 223, 224, 225, 226, 227, 228, 229, 230, 231, 232, 233, 234, 235, 236, 237, 238, 239, 240, 241, 242, 243, 244, 245, 246, 247, 248, 249, 250>, 289>' requested here
      typedef typename make_indices_impl,
                       ^


you can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8876/4388/summary.html

[cmsbuild]

+1
Tested at: 6d2b431
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8876/4435/summary.html

[deguio]

+1