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
HLT menu development #35863
HLT menu development #35863
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-35863/26249
|
A new Pull Request was created by @Martin-Grunewald (Martin Grunewald) 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 |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-6856d7/19987/summary.html Comparison Summary@slava77 comparisons for the following workflows were not done due to missing matrix map:
Summary:
|
Small-but-visible changes in some EGM triggers. |
+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 now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1
|
@Martin-Grunewald @missirol |
Looking into it. If a solution is not identified quickly, would it be better to make a PR to disable the unit test (instead of reverting this PR)? |
For me, the unit test in question, https://github.com/cms-sw/cmssw/blob/master/RecoTauTag/HLTProducers/test/testL2TauTagNN.py seems unacceptable as a unit test as it appears to run the menu itself in a special environment, customised, which now fails as the input menu has changed. This goes way beyond a supposedly simple unit test. So the required course of action is to remove the unit test. |
Okay, I will make a PR to remove the unit test (while trying to figure out if there is any kind of underlying problem). |
PR description:
HLT menu development
Based on CMSSW_12_1_X_2021-10-26-2300
PR validation:
TSG tests
if this PR is a backport please specify the original PR and why you need to backport that PR:
N/A