Removing CSV+SL and adding SL taggers to the default PAT jet configuration

[ferencek]

Given the fact that the CombinedMVA tagger uses SL tagger discriminators as input, it would be beneficial to also have SL taggers stored in MiniAOD to speed up and simplify any future retrainings of ''supercombined" taggers.

@pvmulder @imarches @acaudron

[cmsbuild]

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

Removing CSV+SL and adding SL taggers to the default PAT jet configuration

It involves the following packages:

PhysicsTools/PatAlgos

@cmsbuild, @vadler, @monttj can you please review it and eventually sign? Thanks.
@rappoccio, @imarches, @ahinzmann, @acaudron, @mmarionncern, @jdolen, @nhanvtran, @schoef, @ferencek, @gpetruc, @mariadalfonso, @pvmulder, @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.

[monttj]

@ferencek
I am just curious that we are not using "pfCombinedSecondaryVertexSoftLeptonBJetTags" tagger at the analysis level ?

[monttj]

please test

[cmsbuild]

The tests are being triggered in jenkins.

[cmsbuild]

+1
Tested at: ad05d74
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-10157/4568/summary.html

[ferencek]

@monttj, studies of the performance of CSV+SL indicated there are some not fully understood features in its training (@pvmulder can confirm) and that in the end better performance is obtained from pfCombinedMVABJetTags. The main difference between the two is that CSV+SL takes as input all of the input variables of the CSVv2 tagger and the soft lepton (SL) taggers while pfCombinedMVABJetTags takes as input the output disciminators from the JP, CSVv2, and SL taggers.

[monttj]

+1

[monttj]

@ferencek thank you!

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_5_X IBs once checked with relvals in the development release cycle of CMSSW (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar

[davidlange6]

+1