-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
New Run3 era for updated HB thresholds in 2023 #41271
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-41271/35030
|
A new Pull Request was created by @swagata87 (Swagata Mukherjee) for master. It involves the following packages:
@perrotta, @rappoccio, @clacaputo, @cmsbuild, @mandrenguyen, @fabiocos, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-41271/35032
|
Pull request #41271 was updated. @perrotta, @rappoccio, @clacaputo, @cmsbuild, @mandrenguyen, @fabiocos, @davidlange6 can you please check and sign again. |
FYI also |
@davidlange6 sorry I took the liberty to suggest Swagata to ping you here in order to double check that this is the correct (and best) way to implement the scenario that will eventually be deployed in Tier0. Thanks! |
@cmsbuild please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-67de5d/31853/summary.html Comparison SummarySummary:
|
Just thinking out loud, how about creating setting up all |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-67de5d/31965/summary.html
Comparison SummarySummary:
|
+reconstruction As mentioned here #41271 (comment) |
+1 |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will be automatically merged. |
thanks, so I will prepare a backport to data-taking release (13_0_X) |
PR description:
Creating a new era to update HB thresholds.
This PR is essentially doing similar things as done in #40689 and #40756 for 12_6_X.
PR validation:
Checked with
12434.0
This PR needs to be back-ported to data-taking release 13_0_X.
FYI @cms-sw/hcal-dpg-l2 @cms-sw/pf-l2 @francescobrivio @davidlange6