New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
L1TGlobalPrescaler: additional modes based on the prescales in the EventSetup (8.1.x) #16261
L1TGlobalPrescaler: additional modes based on the prescales in the EventSetup (8.1.x) #16261
Conversation
…entSetup - applyPrescaleValues: apply the given prescale values (original behaviour) - applyPrescaleRatios: apply prescales equal to ratio between the given values and the ones read from the EventSetup - applyColumnValues: apply the prescale values from the EventSetup corresponding to the given column index - applyColumnRatios: apply prescales equal to ratio between the values corresponsing to the given column index, and the ones read from the EventSetup In the last two cases, also sets the prescale column index to the given one.
please test |
The tests are being triggered in jenkins. |
A new Pull Request was created by @fwyzard (Andrea Bocci) for CMSSW_8_1_X. It involves the following packages: L1Trigger/L1TGlobal @cmsbuild, @rekovic, @mulhearn, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here #13028 |
+1 The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: |
Comparison is ready @slava77 comparisons for the following workflows were not done due to missing matrix map:
|
Please note that this module is not used in any central workflows. |
@rekovic - any comments on this PR? |
seems they managed to sign the 80x pr - so I'll merge this one |
Additional modes based on the prescales in the EventSetup:
In the last two cases, also sets the prescale column index to the given one.