-
Notifications
You must be signed in to change notification settings - Fork 4.6k
Additions to scouting DQM with the ScoutingCollectionMonitor and a new ScoutingMuonPropertiesAnalyzer [15_1_X] #49131
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
Additions to scouting DQM with the ScoutingCollectionMonitor and a new ScoutingMuonPropertiesAnalyzer [15_1_X] #49131
Conversation
…ng online DQM client
…uting muons vertexing
…pertiesAnalyzer/Monitoring
|
type ngt |
|
Pull request #49131 was updated. |
|
cms-bot internal usage |
|
test parameters:
|
|
@cmsbuild, please test |
|
A new Pull Request was created by @jprendi for CMSSW_15_1_X. It involves the following packages:
@Martin-Grunewald, @ctarricone, @gabrielmscampos, @mmusich, @nothingface0, @rseidita can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
|
+1 Size: This PR adds an extra 52KB to repository Comparison SummarySummary:
|
6d30a70 to
5a38b44
Compare
|
Pull request #49131 was updated. @Martin-Grunewald, @cmsbuild, @ctarricone, @gabrielmscampos, @mmusich, @nothingface0, @rseidita can you please check and sign again. |
…SVNoVtx_mass_Z plot
|
Pull request #49131 was updated. @Martin-Grunewald, @cmsbuild, @ctarricone, @gabrielmscampos, @mmusich, @nothingface0, @rseidita can you please check and sign again. |
|
please test |
|
+1 Size: This PR adds an extra 44KB to repository Comparison SummarySummary:
|
|
+dqm |
|
+hlt |
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_15_1_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_16_0_X is complete. This pull request will now be reviewed by the release team before it's merged. @sextonkennedy, @mandrenguyen, @ftenchini (and backports should be raised in the release meeting by the corresponding L2) |
|
+1 |
backport of #49130
PR description:
This PR introduces a new DQMEDAnalyzer ScoutingMuonPropertiesAnalyzer for further monitoring of the scouting muon collection complimenting ScoutingCollectionMonitor and additions to the ScoutingCollectionMonitor module.
For
DQM/HLTEvF/plugins/ScoutingCollectionMonitor.ccand inDQM/Integration/python/clients/scouting_dqm_sourceclient-live_cfg.py, we enable monitoring of multiplicity for all the scouting objects but also IP w.r.t beam spot or dxy, dz w.r.t BS.N.B.: compared to the initial #49130 to master, we did not turn off monitoring of the calorechits in <= 2024 in this backport, as the backport of #49084 is not available yet
HLTriggerOffline/Scouting/plugins/ScoutingMuonPropertiesAnalyzer.ccandHLTriggerOffline/Scouting/python/ScoutingMuonPropertiesMonitoring_cfi.pyis new for the scouting muon properties analyzer.DQM/HLTEvF/python/ScoutingMuonMonitoring_cff.py,DQMOffline/HLTScouting/python/HLTScoutingDqmOffline_cff.pyandHLTriggerOffline/Scouting/plugins/BuildFile.xmlwere adapted to include the ScoutingMuonPropertiesAnalyzer.The scouting muon properties analyzer is based of the work done by @prijb here.
PR validation:
We started out with:
and validated via using the input files at cms-data/DQM-Integration#12 by copying them inside the
DQM/Integration/data/run392642folder and then running:followed by subsquent inspection of the output histograms.
We also tested the workflow via:
Running
git cms-checkdeps -a -Ashows further dependencies toDQMOffline/Configuration.If this PR is a backport please specify the original PR and why you need to backport that PR. If this PR will be backported please specify to which release cycle the backport is meant for:
Backport of #49130 to
15_1_X