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
Synchronisation of HCal Config for HLT_75E33 menu (Phase2) #39835
Synchronisation of HCal Config for HLT_75E33 menu (Phase2) #39835
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-39835/32715
|
A new Pull Request was created by @beaucero for master. It involves the following packages:
@cmsbuild, @missirol, @Martin-Grunewald can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
test parameters:
|
please test |
@beaucero, shouldn't the same update be applied to What is the plan to resolve these duplications, and use only the modules named |
-1 Failed Tests: RelVals-INPUT RelVals-INPUTThe relvals timed out after 4 hours. Comparison Summary@slava77 comparisons for the following workflows were not done due to missing matrix map:
Summary:
|
Actually I don't see here one more suggested/considered (back in September) "saving time" change: (as it's in HLTrigger/Configuration/python/HLT_2022v14_cff.py) |
@missirol Sorry for the delay, loking further indeed, there are quite some duplications of modules which should be removed etc. But we are currently busy with the Annual Review, so cleaning will come later on. Looks likethe second file should be also modified (until it disappears) @abdoulline Sorry for missing the suggestions. |
@beaucero , okay, thanks. It looks okay now. For updates like this to the HLT Phase-2 menu, what is currently the target release? This PR updates 12_6_X; for 12_5_X or lower, separate PRs would be needed (after this one is integrated). |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-39835/32725
|
Pull request #39835 was updated. @cmsbuild, @missirol, @Martin-Grunewald can you please check and sign again. |
@missirol, no real target as the above changes are not really changing the results of the HLT. I was simply a mis-synchronisation of files which was not nice to carry on propagating. So what is easier to go, we should go for this one. (Same thing when we will do the cleaning, there will be no real release target etc) |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-1be434/28499/summary.html Comparison Summary@slava77 comparisons for the following workflows were not done due to missing matrix map:
Summary:
|
+hlt |
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 now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
Synchronisation of HCal Configuration for HLT_75E33 menu, the tiny changes in the config are not changing physics results
PR validation:
Running HLT_Simplified_menu Phase2 with the new configuration is successful.