Save L1 stage2 info in offline data tiers (80X)

[gpetruc]

80X version of #14051

[cmsbuild]

A new Pull Request was created by @gpetruc (Giovanni Petrucciani) for CMSSW_8_0_X.

It involves the following packages:

L1Trigger/Configuration
PhysicsTools/PatAlgos

@cvuosalo, @cmsbuild, @rekovic, @slava77, @mulhearn, @davidlange6 can you please review it and eventually sign? Thanks.
@TaiSakuma, @rappoccio, @imarches, @ahinzmann, @acaudron, @mmarionncern, @Martin-Grunewald, @jdolen, @nhanvtran, @schoef, @ferencek, @mariadalfonso, @pvmulder this is something you requested to watch as well.
@slava77, @Degano, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

[slava77]

@cmsbuild please test

[cmsbuild]

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

[cmsbuild]

+1
Tested at: af26b2c
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-14068/12375/summary.html

[cmsbuild]

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

The workflows 1003.0, 1000.0 have different files in step1_dasquery.log than the ones found in the baseline. You may want to check and retrigger the tests if necessary. You can check it in the "files" directory in the results of the comparisons

[slava77]

+1

for #14068 af26b2c

• changes are in line with the description; the same code change as in Save L1 stage2 info in offline data tiers (81X) #14051
• jenkins tests pass: comparisons with baseline show no difference as expected; 2017 workflows now contain the stage2 collections on output
  in 10021.0 reco file (10 events of TenMuE_0_200)

-----------------------------------------------------------------
   or, B         new, B      delta, B   delta, %   deltaJ, %    branch 
-----------------------------------------------------------------
      0.0 ->       285.6        286     NEWO   0.05     l1tEtSumBXVector_caloStage2Digis_EtSum_RECO.
      0.0 ->       325.5        326     NEWO   0.05     l1tJetBXVector_caloStage2Digis_Jet_RECO.
      0.0 ->       290.4        290     NEWO   0.05     l1tTauBXVector_caloStage2Digis_Tau_RECO.
      0.0 ->       252.0        252     NEWO   0.04     GlobalExtBlkBXVector_gtStage2Digis__RECO.
      0.0 ->       286.1        286     NEWO   0.05     GlobalAlgBlkBXVector_gtStage2Digis__RECO.
      0.0 ->       438.6        439     NEWO   0.07     l1tMuonBXVector_gmtStage2Digis_Muon_RECO.
      0.0 ->       252.5        252     NEWO   0.04     l1tEGammaBXVector_caloStage2Digis_EGamma_RECO.
-------------------------------------------------------------
   618395 ->      620525       2130             0.3     ALL BRANCHES

[slava77]

@davidlange6
this could have gone to 805
I tested the 81X version of this change on run2 data with T0 config before signing these PRs

[mulhearn]

+1

[mulhearn]

Thanks for doing this!

[davidlange6]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_8_0_X IBs (tests are also fine). This pull request will be automatically merged.