-
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
Definition of a new DQM Offline sequence to be run on Express Stream #41944
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-41944/35913
|
A new Pull Request was created by @rvenditti for master. It involves the following packages:
@perrotta, @cmsbuild, @rappoccio, @nothingface0, @emanueleusai, @davidlange6, @pmandrik, @syuvivida, @tjavaid, @micsucmed, @fabiocos, @rvenditti 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-2f2e61/33134/summary.html Comparison SummaryThere are some workflows for which there are errors in the baseline: Summary:
|
+1 |
+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. |
PR description:
The goal of this PR is to define a new sequence in DQM Offline, that should be run on the Express Stream. The new sequence will produce a reduced number of ME (10% less) wrt the DQM sequence that is presently run on Express (the "standardDQM" sequence).
By this way, also the number of ME that should be harvested in step3 will be reduced (thanks to a new ad-hoc harvesting sequence), and hopefully this would mitigate the memory issues observed at T0 during 2022 and 2023 collisions (discussed here #38976).
The subsystem sequences that will be run with this new configuration, are decided on the basis of the feedback received during the DQM General meeting https://indico.cern.ch/event/1259866/. Wrt the old sequence, we removed DT, EGamma, HCAL, JetMet, L1T and Physics. Additional room for improvement: remove MuonPOG (not yet done, since part of the CSC plots are produced in the MuonPOG sequence).
PR validation:
A generic test that mimic the offline reconstruction+ DQM steps on top of the raw express data has been performed with
cmsDriver.py step2 --conditions 130X_dataRun3_Express_v2 --data --datatier RECO,DQMIO --era Run3 --eventcontent RECO,DQM --filein file:/eos/cms/store/t0streamer/Data/Express/000/368/454/run368454_ls0134_streamExpress_StorageManager.dat --fileout file:step2_modified.root --nStreams 2 --nThreads 8 --no_exec --number 10 --process RECO,DQM --python_filename step2_modified_cfg.py --scenario pp --step RAW2DIGI,L1Reco,RECO,DQM:@standardDQMExpress
cmsDriver.py step3 --conditions 130X_dataRun3_Express_v2 --data --era Run3 --filein step2_modified_inDQM.root --fileout file:step3_modified.root --filetype DQM --nStreams 2 --no_exec --number 10 --python_filename step_3_modified_cfg.py --scenario pp --step HARVESTING:@standardDQMExpress
The DQMIO files produced from the old (standardDQM) and new (standardDQMExpress) workflow have been inspected with dqmiodumpmetadata.py and dqmiolistmes.py scripts.
This PR should be backported to 13_0_X, because we target to have the new sequence running at T0.