supply DQM for `SiPixelCalSingleMuonTight` #39382

mmusich · 2022-09-13T11:00:28Z

PR description:

With the foreseen increase of isolated muon trigger rates (see https://its.cern.ch/jira/browse/CMSHLT-2465 ) it becomes possible to run the SiPixelCalSingleMuonTight at express, allowing for quasi-online efficiency monitoring.
This PR adds a stub of DQM monitor for that AlCa flavour to start producing basic cluster properties histograms that are used as ingredients of the hit efficiency measurement.

PR validation:

Run

 runTheMatrix.py -l 1002.4 -t 4 -j 8 --iboes

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:

Not a backport, but will need to be backported

mmusich · 2022-09-13T11:00:43Z

type trk, new-feature

mmusich · 2022-09-13T11:01:00Z

test parameters:

workflow = 1002.4

cmsbuild · 2022-09-13T11:09:16Z

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-39382/32085

This PR adds an extra 24KB to repository

cmsbuild · 2022-09-13T11:09:38Z

A new Pull Request was created by @mmusich (Marco Musich) for master.

It involves the following packages:

Configuration/StandardSequences (operations)
DQMOffline/CalibTracker (dqm)
DQMOffline/Configuration (dqm)

@perrotta, @rappoccio, @pmandrik, @emanueleusai, @ahmad3213, @cmsbuild, @jfernan2, @syuvivida, @davidlange6, @micsucmed, @fabiocos, @rvenditti can you please review it and eventually sign? Thanks.
@fabiocos, @VourMa, @makortel, @felicepantaleo, @GiacomoSguazzoni, @JanFSchulte, @rovere, @VinInn, @Martin-Grunewald, @missirol, @lecriste, @mtosi, @mmusich, @threus, @dgulhan, @slomeo, @rociovilar this is something you requested to watch as well.
@perrotta, @dpiparo, @rappoccio you are the release manager for this.

cms-bot commands are listed here

mmusich · 2022-09-13T11:12:21Z

@cmsbuild, please test

cmsbuild · 2022-09-13T17:41:16Z

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-b00a7f/27500/summary.html
COMMIT: bafba68
CMSSW: CMSSW_12_6_X_2022-09-13-1100/el8_amd64_gcc10
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/39382/27500/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

@slava77 comparisons for the following workflows were not done due to missing matrix map:

/data/cmsbld/jenkins/workspace/compare-root-files-short-matrix/data/PR-b00a7f/1002.4_RunDoubleMuon2022B+RunDoubleMuon2022B+TIER0PROMPTRUN3+HARVESTPROMPTRUN3

Summary:

No significant changes to the logs found
Reco comparison results: 0 differences found in the comparisons
DQMHistoTests: Total files compared: 52
DQMHistoTests: Total histograms compared: 3720269
DQMHistoTests: Total failures: 2
DQMHistoTests: Total nulls: 2
DQMHistoTests: Total successes: 3720243
DQMHistoTests: Total skipped: 22
DQMHistoTests: Total Missing objects: 0
DQMHistoSizes: Histogram memory added: 4.926 KiB( 51 files compared)
DQMHistoSizes: changed ( 1000.0,... ): 2.455 KiB AlCaReco/SiPixelCalSingleMuonTight
DQMHistoSizes: changed ( 1000.0,... ): 0.004 KiB MessageLogger/Errors
DQMHistoSizes: changed ( 1000.0,... ): 0.004 KiB MessageLogger/Warnings
Checked 215 log files, 49 edm output root files, 52 DQM output files
TriggerResults: found differences in 2 / 51 workflows

emanueleusai · 2022-09-14T13:29:04Z

+1

DQMOffline/CalibTracker/plugins/SiPixelCalSingleMuonAnalyzer.cc

mmusich · 2022-09-15T07:47:25Z

@cmsbuild, please test

cmsbuild · 2022-09-15T11:46:58Z

-1

Failed Tests: UnitTests
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-b00a7f/27553/summary.html
COMMIT: dbb71e4
CMSSW: CMSSW_12_6_X_2022-09-14-2300/el8_amd64_gcc10
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/39382/27553/install.sh to create a dev area with all the needed externals and cmssw changes.

Unit Tests

I found errors in the following unit tests:

---> test test-das-selected-lumis had ERRORS

Comparison Summary

@slava77 comparisons for the following workflows were not done due to missing matrix map:

/data/cmsbld/jenkins/workspace/compare-root-files-short-matrix/data/PR-b00a7f/1002.4_RunDoubleMuon2022B+RunDoubleMuon2022B+TIER0PROMPTRUN3+HARVESTPROMPTRUN3

Summary:

No significant changes to the logs found
Reco comparison results: 6 differences found in the comparisons
DQMHistoTests: Total files compared: 52
DQMHistoTests: Total histograms compared: 3720269
DQMHistoTests: Total failures: 11
DQMHistoTests: Total nulls: 2
DQMHistoTests: Total successes: 3720234
DQMHistoTests: Total skipped: 22
DQMHistoTests: Total Missing objects: 0
DQMHistoSizes: Histogram memory added: 4.926 KiB( 51 files compared)
DQMHistoSizes: changed ( 1000.0,... ): 2.455 KiB AlCaReco/SiPixelCalSingleMuonTight
DQMHistoSizes: changed ( 1000.0,... ): 0.004 KiB MessageLogger/Errors
DQMHistoSizes: changed ( 1000.0,... ): 0.004 KiB MessageLogger/Warnings
Checked 215 log files, 49 edm output root files, 52 DQM output files
TriggerResults: found differences in 2 / 51 workflows

mmusich · 2022-09-15T11:48:17Z

found errors in the following unit tests:
---> test test-das-selected-lumis had ERRORS

test failure is unrelated.

mmusich · 2022-09-15T11:48:24Z

@cmsbuild, please test

cmsbuild · 2022-09-15T16:32:10Z

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-b00a7f/27567/summary.html
COMMIT: dbb71e4
CMSSW: CMSSW_12_6_X_2022-09-14-2300/el8_amd64_gcc10
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/39382/27567/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

@slava77 comparisons for the following workflows were not done due to missing matrix map:

/data/cmsbld/jenkins/workspace/compare-root-files-short-matrix/data/PR-b00a7f/1002.4_RunDoubleMuon2022B+RunDoubleMuon2022B+TIER0PROMPTRUN3+HARVESTPROMPTRUN3

Summary:

No significant changes to the logs found
Reco comparison results: 2 differences found in the comparisons
DQMHistoTests: Total files compared: 52
DQMHistoTests: Total histograms compared: 3720269
DQMHistoTests: Total failures: 8
DQMHistoTests: Total nulls: 2
DQMHistoTests: Total successes: 3720237
DQMHistoTests: Total skipped: 22
DQMHistoTests: Total Missing objects: 0
DQMHistoSizes: Histogram memory added: 4.926 KiB( 51 files compared)
DQMHistoSizes: changed ( 1000.0,... ): 2.455 KiB AlCaReco/SiPixelCalSingleMuonTight
DQMHistoSizes: changed ( 1000.0,... ): 0.004 KiB MessageLogger/Errors
DQMHistoSizes: changed ( 1000.0,... ): 0.004 KiB MessageLogger/Warnings
Checked 215 log files, 49 edm output root files, 52 DQM output files
TriggerResults: found differences in 2 / 51 workflows

perrotta · 2022-09-15T17:34:35Z

@emanueleusai , the changes since your previous signature are rather minor, and do not affect the results, I keep that signature for good and merge this PR in the master, aiming at merging its backports by tomorrow.

perrotta · 2022-09-15T17:34:41Z

+1

perrotta · 2022-09-15T17:35:20Z

merge

emanueleusai · 2022-09-17T00:11:07Z

+1

cmsbuild · 2022-09-17T00:11:28Z

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.

cmsbuild added this to the CMSSW_12_6_X milestone Sep 13, 2022

cmsbuild added code-checks-pending dqm-pending new-feature operations-pending orp-pending pending-signatures tests-pending trk labels Sep 13, 2022

cmsbuild added code-checks-approved and removed code-checks-pending labels Sep 13, 2022

cmsbuild added tests-started and removed tests-pending labels Sep 13, 2022

This was referenced Sep 13, 2022

[12.5.X] supply DQM for SiPixelCalSingleMuonTight #39385

Merged

[12.4.X] supply DQM for SiPixelCalSingleMuonTight #39386

Merged

cmsbuild added tests-approved and removed tests-started labels Sep 13, 2022

cmsbuild added dqm-approved and removed dqm-pending labels Sep 14, 2022

perrotta reviewed Sep 14, 2022

View reviewed changes

supply DQM for SiPixelCalSingleMuonTight

dbb71e4

mmusich force-pushed the SiPixelCalSingleMuonTightDQM branch from bafba68 to dbb71e4 Compare September 15, 2022 07:37

cmsbuild removed the dqm-approved label Sep 15, 2022

cmsbuild added tests-started and removed tests-pending labels Sep 15, 2022

cmsbuild added tests-rejected and removed tests-started labels Sep 15, 2022

cmsbuild added tests-started and removed tests-rejected labels Sep 15, 2022

cmsbuild added tests-approved and removed tests-started labels Sep 15, 2022

cmsbuild added operations-approved orp-approved and removed operations-pending orp-pending labels Sep 15, 2022

cmsbuild merged commit 017a0b9 into cms-sw:master Sep 15, 2022

mmusich deleted the SiPixelCalSingleMuonTightDQM branch September 16, 2022 07:18

cmsbuild added dqm-approved fully-signed and removed dqm-pending pending-signatures labels Sep 17, 2022

francescobrivio mentioned this pull request Sep 19, 2022

Replay testing CMSSW_12_4_9 dmwm/T0#4753

Closed

makortel mentioned this pull request Sep 20, 2022

Crashes in workflow 39434.911 #39445

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

supply DQM for `SiPixelCalSingleMuonTight` #39382

supply DQM for `SiPixelCalSingleMuonTight` #39382

mmusich commented Sep 13, 2022

mmusich commented Sep 13, 2022

mmusich commented Sep 13, 2022

cmsbuild commented Sep 13, 2022

cmsbuild commented Sep 13, 2022

mmusich commented Sep 13, 2022

cmsbuild commented Sep 13, 2022

emanueleusai commented Sep 14, 2022

mmusich commented Sep 15, 2022

cmsbuild commented Sep 15, 2022

mmusich commented Sep 15, 2022

mmusich commented Sep 15, 2022

cmsbuild commented Sep 15, 2022

perrotta commented Sep 15, 2022

perrotta commented Sep 15, 2022

perrotta commented Sep 15, 2022

emanueleusai commented Sep 17, 2022

cmsbuild commented Sep 17, 2022

supply DQM for SiPixelCalSingleMuonTight #39382

supply DQM for SiPixelCalSingleMuonTight #39382

Conversation

mmusich commented Sep 13, 2022

PR description:

PR validation:

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:

mmusich commented Sep 13, 2022

mmusich commented Sep 13, 2022

cmsbuild commented Sep 13, 2022

cmsbuild commented Sep 13, 2022

mmusich commented Sep 13, 2022

cmsbuild commented Sep 13, 2022

Comparison Summary

emanueleusai commented Sep 14, 2022

mmusich commented Sep 15, 2022

cmsbuild commented Sep 15, 2022

Unit Tests

Comparison Summary

mmusich commented Sep 15, 2022

mmusich commented Sep 15, 2022

cmsbuild commented Sep 15, 2022

Comparison Summary

perrotta commented Sep 15, 2022

perrotta commented Sep 15, 2022

perrotta commented Sep 15, 2022

emanueleusai commented Sep 17, 2022

cmsbuild commented Sep 17, 2022

supply DQM for `SiPixelCalSingleMuonTight` #39382

supply DQM for `SiPixelCalSingleMuonTight` #39382