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
[10_2_X] OfflineDQM: Offline L1T muon DQM bugfix for muon resolution plots CMSLITDPG-763 #24874
Conversation
A new Pull Request was created by @stahlleiton (Andre Stahl) for CMSSW_10_2_X. It involves the following packages: DQMOffline/L1Trigger @cmsbuild, @andrius-k, @kmaeshima, @schneiml, @nsmith-, @rekovic, @jfernan2, @thomreis can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
+1 |
The tests are being triggered in jenkins. |
-1 Tested at: fd35d2b You can see the results of the tests here: I found follow errors while testing this PR Failed tests: RelVals
When I ran the RelVals I found an error in the following worklfows: runTheMatrix-results/250202.181_TTbar_13UP18+TTbar_13UP18+PREMIXUP18_PU25+DIGIPRMXLOCALUP18_PU25+RECOPRMXUP18_PU25+HARVESTUP18_PU25/step5_TTbar_13UP18+TTbar_13UP18+PREMIXUP18_PU25+DIGIPRMXLOCALUP18_PU25+RECOPRMXUP18_PU25+HARVESTUP18_PU25.log |
Comparison not run due to runTheMatrix errors (RelVals and Igprof tests were also skipped) |
please test error looks unrelated to the code changes. |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_10_2_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_10_4_X is complete. This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
Hi @stahlleiton, as 10_2 is no longer being used maybe this PR should be closed? |
@stahlleiton @rekovic is this PR still really needed now that 10_2_X is no more used for data taking? |
hold |
Pull request has been put on hold by @fabiocos |
Hi @fabiocos |
@thomreis pp data taking is over and for HI 10_3_X is used. Of course MC production is ongoign as well as analysis, but you do not need this for those activities, right? So I would just close it. |
Agreed. |
Backport of #24870