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
[Backport] Updates related to the L1 menu v1_1_0 for the start of Run 3 #38331
[Backport] Updates related to the L1 menu v1_1_0 for the start of Run 3 #38331
Conversation
A new Pull Request was created by @elfontan (Elisa Fontanesi) for CMSSW_12_4_X. It involves the following packages:
@epalencia, @cmsbuild, @cecilecaillol, @rekovic can you please review it and eventually sign? Thanks. cms-bot commands are listed here
|
please test with cms-sw/cmsdist#7926 |
-1 Failed Tests: UnitTests Unit TestsI found errors in the following unit tests: ---> test L1O2O_L1TMenu had ERRORS ---> test L1O2O_L1TSubs had ERRORS Comparison SummarySummary:
|
test parameters:
|
please test with cms-sw/cmsdist#7926 Let's see if #38331 (comment) helps (I encountered the same issue locally; looks like the new L1T unit test expects the other pkg to be checked out locally). What I'm missing is why the unit test did not run in the original PR (here); |
backport of #38250 |
please abort |
please test My bad, I had to fix #38331 (comment). |
From this PR I see [1], while from the original PR I see [2]. I'm not sure why. Naively, I would expect [1] even in [1] https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/25498/consoleText + git cms-checkdeps -A -a
>> Checking L1Trigger/Configuration CMSSW_12_4_X_2022-06-13-1100
x L1Trigger/Configuration/python/customiseUtils.py
>> Checking L1Trigger/L1TGlobal CMSSW_12_4_X_2022-06-13-1100
x L1Trigger/L1TGlobal/python/PrescalesVetosFract_cff.py
x L1Trigger/L1TGlobal/python/PrescalesVetos_cff.py
x L1Trigger/L1TGlobal/test/runGlobalFakeInputProducer.py
Checking out these packages: 1
L1TriggerConfig/L1TConfigProducers (python) [2] https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/25288/consoleText + git cms-checkdeps -A -a
>> Checking L1Trigger/Configuration CMSSW_12_5_X_2022-06-05-0000
x L1Trigger/Configuration/python/customiseUtils.py
>> Checking L1Trigger/L1TGlobal CMSSW_12_5_X_2022-06-05-0000
x L1Trigger/L1TGlobal/python/PrescalesVetosFract_cff.py
x L1Trigger/L1TGlobal/python/PrescalesVetos_cff.py
x L1Trigger/L1TGlobal/test/runGlobalFakeInputProducer.py
Checking out these packages: 0 |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-14cc25/25498/summary.html Comparison SummarySummary:
|
+l1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_12_4_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_12_5_X is complete. 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
|
Hi @smuzaffar , just for my understanding, would you mind having a look at #38331 (comment) , and explaining if this is the expected behaviour of the PR tests? It seems that changes in |
@missirol , looks like the python dependency check is broken in 12.5.X as the dependency files in 12.5.X are
while in 12.4.X we have
I am checking what broken it |
@missirol , cms-sw/cmsdist#7932 shoudl fix the python dependency issue |
Thanks, @smuzaffar ! |
PR description:
Backport of #38250 to CMSSW_12_4_0.
A full set of xml files for the L1 emulation of prescales and masks coherent with the updated L1 menu for Run 3 has been created and pushed to L1Trigger-L1TGlobal (PR#11). A backport of these files has been requested in the github issue cms-data/L1Trigger-L1TGlobal#12. Changes will be backported to CMSSW_12_4_0 in cms-sw/cmsdist after the tests: PR#7926 for 12_4_X cms-dist ready.
PR validation:
Basic tests performed successfully starting from CMSSW_12_4_X_2022-06-10-2300.