L1t run l1t famos bugfix 730

[mulhearn]

This fixes a bug when running Fast Simulation in postLS1 scenario, where new L1T emulation was not run but inputs were expected downstream. This bug fix causes new L1T emulation to be run in all scenarios where legacy GCT emulation was run, if postLS1 customization is made. It also sets the l1extra producer to use simulated or unpacked digis based on context.

[cmsbuild]

A new Pull Request was created by @mulhearn for CMSSW_7_3_X.

L1t run l1t famos bugfix 730

It involves the following packages:

L1Trigger/L1TCommon

@cmsbuild, @nclopezo, @mulhearn can you please review it and eventually sign? Thanks.
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.
@nclopezo you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[perrotta]

Great @mulhearn : it does its job!
FYI: with this PR no more crashes in any of the fastsim addOn's run for the HLT tests, nor in the short matrix.

[perrotta]

@mulhearn: can you please sign?
@nclopezo: this bug fix avoids fastsim crashes as the ones with the current 73X IB

[mulhearn]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_3_X IBs unless changes or unless it breaks tests. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @nclopezo, @ktf, @smuzaffar

[cmsbuild]

+1
Tested at: b6b9f61
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-6597/1169/summary.html

[cmsbuild]

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

[cmsbuild]

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

[cmsbuild]

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

[davidlange6]

+1