Remove legacy b-taggers from the standard reco

[ferencek]

No description provided.

[cmsbuild]

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

Remove legacy b-taggers from the standard reco

It involves the following packages:

DQMOffline/RecoB
HLTriggerOffline/Higgs
PhysicsTools/PatAlgos
RecoBTag/Configuration

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

[cmsbuild]

The tests are being triggered in jenkins.

[cmsbuild]

+1
Tested at: ae2cd93
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-11198/7977/summary.html

[cmsbuild]

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

[slava77]

Here and in the rest of this PR, please remove the code, not comment it out.

[ferencek]

Done.

[slava77]

@cmsbuild please test

[cmsbuild]

The tests are being triggered in jenkins.

[slava77]

DQM is showing empty plots in the following folders

• TopHLTOffline/Top/SemiMuoninc/step0
• TopHLTOffline/Top/SemiMuoninc/step1
• TopHLTOffline/Top/SemiMuoninc/step2
  I suppose these are symptoms of the same problem.

Things seem good on all other comparisons or technical values I have looked at.

[ferencek]

OK, I found and fixed a typo in the tagger name in top HLT DQM. Hopefully all the remaining problems are now resolved.

[cmsbuild]

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

[deguio]

please test

[cmsbuild]

The tests are being triggered in jenkins.

[cmsbuild]

+1
Tested at: 8d171b5
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-11198/8072/summary.html

[cmsbuild]

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

[slava77]

+1

for #11198 8d171b5

• code/config changes are OK, including planned removal of the legacy b-taggers and also removal of TagInfos from EI event content
• jenkins tests pass and comparisons with baseline show expected changes
  • fwlite-based comparisons show "empty" plots corresponding to removed products
  • DQM/validation plots show some changes in b-tagging related distributions where the legacy taggers were replaced with the current pf candidate based taggers
• tested locally in CMSSW_7_6_X_2015-09-07-2300 /test area sign589/ to check the technical performance and confirm jenkins tests on higher statistics
  • about 30 tagger producers are no longer running (reco+PAT workflow); this has low impact on total CPU and memory (40ms/event or ~0.3% of RECO)
  • in RECO/AOD one new tagging variable was added pfCombinedInclusiveSecondaryVertexV2BJetTagsEI and 21/16 legacy taggers were removed with a net effect of 1.4% reduction in AOD and 1% reduction of RECO size

[deguio]

+1