Updated soft electron dR cut to better match the jet size

[ferencek]

Updating the soft electron dR cut from 0.5 to 0.4 to better match the jet size.

[cmsbuild]

A new Pull Request was created by @ferencek (Dinko Ferencek) for CMSSW_7_4_X.

Updated soft electron dR cut to better match the jet size

It involves the following packages:

RecoBTag/SoftLepton

@cmsbuild, @cvuosalo, @nclopezo, @slava77 can you please review it and eventually sign? Thanks.
@ferencek, @acaudron, @pvmulder, @imarches 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
we also need a 75X version of this

Also, this feature looks more like an improvement than a normal bugfix. Is it fixing some issue found during 74X validation?

[cmsbuild]

The tests are being triggered in jenkins.

[ferencek]

The soft muon and soft electron taggers select leptons in different ways. While muons are selected from the jet constituents, electrons are selected by matching gedGsfElectrons to jets. So the soft muon tagger does not apply an explicit dR requirement but its effective value is 0.4 (ignoring special cases of overlapping jets, etc.). This PR therefore makes the lepton selection more consistent between the two taggers. In terms of the performance, it is expected to get slightly worse for low-pT jets. So this PR probably goes more into the category of bugfixes than performance improvements.

A 75X version will follow.

[cmsbuild]

-1
Tested at: ce5e5aa
The relvals timed out after 2 hours.

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

[cmsbuild]

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

[cmsbuild]

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

[cmsbuild]

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

[davidlange6]

+1