Add reco::Muon displacedMuons to offline validation sequences
PR description:
In this PR we add the code to include the reco::Muon displaced muon AOD collection to the Muon offline validation. An analyzer is added to Validation/RecoMuon/src, very similar to the one for standard muons. Also the necessary configuration files are added, and the sequences included in the global configuration. Validation for standard muon collection is left unchanged.
See presentation on MUO POG General meeting: https://indico.cern.ch/event/1389920/#2-displaced-muon-in-the-valida
PR validation:
New offline DQM has been run on a real validation sample: workflow 12524.0 DisplacedSUSY with 5000 events. The code runs smoothly and the new validation folder Muons/RecoDisplacedMuonV is in the DQM file with the correct plots. As an example, see Dxy plot of reco tracks for dSA [1] and dGl [2].
[1]
[2]
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:
It is not necessary to backport this PR.
cms-bot internal usage
+code-checks
Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-44353/39391
- This PR adds an extra 36KB to repository
A new Pull Request was created by @24LopezR for master.
It involves the following packages:
- Validation/RecoMuon (dqm)
@cmsbuild, @nothingface0, @syuvivida, @antoniovagnerini, @tjavaid, @rvenditti can you please review it and eventually sign? Thanks. @JanFSchulte, @missirol, @CeliaFernandez, @jhgoh, @cericeci, @calderona, @HuguesBrun, @andrea21z, @abbiendi, @rociovilar, @Fedespring, @trocino this is something you requested to watch as well. @antoniovilela, @sextonkennedy, @rappoccio you are the release manager for this.
cms-bot commands are listed here
please test
-1
Failed Tests: RelVals RelVals-INPUT AddOn
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-111b89/37999/summary.html
COMMIT: 334a36cdfefd0eb6c22753cac7a09513a94853ca
CMSSW: CMSSW_14_1_X_2024-03-08-1100/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/44353/37999/install.sh to create a dev area with all the needed externals and cmssw changes.
RelVals
----- Begin Fatal Exception 08-Mar-2024 18:34:05 CET-----------------------
An exception of category 'ProductNotFound' occurred while
[0] Processing Event run: 1 lumi: 1 event: 1 stream: 0
[1] Running path 'validation_step'
[2] Calling method for module RecoDisplacedMuonValidator/'recoDisplacedMuonVMuAssoc_tgt'
Exception Message:
Principal::getByToken: Found zero products matching all criteria
Looking for a container with elements of type: reco::Muon
Looking for module label: displacedMuons
Looking for productInstanceName:
Additional Info:
[a] If you wish to continue processing events after a ProductNotFound exception,
add "TryToContinue = cms.untracked.vstring('ProductNotFound')" to the "options" PSet in the configuration.
----- End Fatal Exception -------------------------------------------------
----- Begin Fatal Exception 08-Mar-2024 18:34:08 CET-----------------------
An exception of category 'ProductNotFound' occurred while
[0] Processing Event run: 1 lumi: 1 event: 1 stream: 0
[1] Running path 'validation_step'
[2] Calling method for module RecoDisplacedMuonValidator/'recoDisplacedMuonVMuAssoc_tgt'
Exception Message:
Principal::getByToken: Found zero products matching all criteria
Looking for a container with elements of type: reco::Muon
Looking for module label: displacedMuons
Looking for productInstanceName:
Additional Info:
[a] If you wish to continue processing events after a ProductNotFound exception,
add "TryToContinue = cms.untracked.vstring('ProductNotFound')" to the "options" PSet in the configuration.
----- End Fatal Exception -------------------------------------------------
----- Begin Fatal Exception 08-Mar-2024 18:54:03 CET-----------------------
An exception of category 'ProductNotFound' occurred while
[0] Processing Event run: 1 lumi: 1 event: 1 stream: 0
[1] Running path 'validation_step'
[2] Calling method for module RecoDisplacedMuonValidator/'recoDisplacedMuonVMuAssoc_tgt'
Exception Message:
Principal::getByToken: Found zero products matching all criteria
Looking for a container with elements of type: reco::Muon
Looking for module label: displacedMuons
Looking for productInstanceName:
Additional Info:
[a] If you wish to continue processing events after a ProductNotFound exception,
add "TryToContinue = cms.untracked.vstring('ProductNotFound')" to the "options" PSet in the configuration.
----- End Fatal Exception -------------------------------------------------
RelVals-INPUT
- 13234.0
13234.0_TTbar_14TeV+2021FS/step2_TTbar_14TeV+2021FS.log - 14034.0
14034.0_TTbar_14TeV+2023FS/step2_TTbar_14TeV+2023FS.log - 13434.0
13434.0_TTbar_14TeV+2021FSPU/step2_TTbar_14TeV+2021FSPU.log
Expand to see more relval errors ...
AddOn Tests
----- Begin Fatal Exception 08-Mar-2024 18:35:42 CET-----------------------
An exception of category 'ProductNotFound' occurred while
[0] Processing Event run: 1 lumi: 1 event: 2 stream: 2
[1] Running path 'validation_step'
[2] Calling method for module RecoDisplacedMuonValidator/'recoDisplacedMuonVMuAssoc_tgt'
Exception Message:
Principal::getByToken: Found zero products matching all criteria
Looking for a container with elements of type: reco::Muon
Looking for module label: displacedMuons
Looking for productInstanceName:
Additional Info:
[a] If you wish to continue processing events after a ProductNotFound exception,
add "TryToContinue = cms.untracked.vstring('ProductNotFound')" to the "options" PSet in the configuration.
----- End Fatal Exception -------------------------------------------------
----- Begin Fatal Exception 08-Mar-2024 18:35:41 CET-----------------------
An exception of category 'ProductNotFound' occurred while
[0] Processing Event run: 1 lumi: 1 event: 4 stream: 2
[1] Running path 'validation_step'
[2] Calling method for module RecoDisplacedMuonValidator/'recoDisplacedMuonVMuAssoc_tgt'
Exception Message:
Principal::getByToken: Found zero products matching all criteria
Looking for a container with elements of type: reco::Muon
Looking for module label: displacedMuons
Looking for productInstanceName:
Additional Info:
[a] If you wish to continue processing events after a ProductNotFound exception,
add "TryToContinue = cms.untracked.vstring('ProductNotFound')" to the "options" PSet in the configuration.
----- End Fatal Exception -------------------------------------------------
----- Begin Fatal Exception 08-Mar-2024 18:35:41 CET-----------------------
An exception of category 'ProductNotFound' occurred while
[0] Processing Event run: 1 lumi: 1 event: 1 stream: 3
[1] Running path 'validation_step'
[2] Calling method for module RecoDisplacedMuonValidator/'recoDisplacedMuonVMuAssoc_tgt'
Exception Message:
Principal::getByToken: Found zero products matching all criteria
Looking for a container with elements of type: reco::Muon
Looking for module label: displacedMuons
Looking for productInstanceName:
Additional Info:
[a] If you wish to continue processing events after a ProductNotFound exception,
add "TryToContinue = cms.untracked.vstring('ProductNotFound')" to the "options" PSet in the configuration.
----- End Fatal Exception -------------------------------------------------
Apparently, I added the validation sequence to fastsim and this should not be there. I fixed it and rerun the tests and now they run OK.
+code-checks
Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-44353/39416
- This PR adds an extra 36KB to repository
Pull request #44353 was updated. @nothingface0, @syuvivida, @cmsbuild, @tjavaid, @antoniovagnerini, @rvenditti can you please check and sign again.
please test
+1
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-111b89/38067/summary.html
COMMIT: 2785e6ec7a2c63601f15c445f68d05dfdf53dbc3
CMSSW: CMSSW_14_1_X_2024-03-12-1100/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/44353/38067/install.sh to create a dev area with all the needed externals and cmssw changes.
Comparison Summary
Summary:
- You potentially added 4 lines to the logs
- Reco comparison results: 63 differences found in the comparisons
- DQMHistoTests: Total files compared: 48
- DQMHistoTests: Total histograms compared: 3297383
- DQMHistoTests: Total failures: 1358
- DQMHistoTests: Total nulls: 0
- DQMHistoTests: Total successes: 3296005
- DQMHistoTests: Total skipped: 20
- DQMHistoTests: Total Missing objects: 0
- DQMHistoSizes: Histogram memory added: 29848.476 KiB( 47 files compared)
- DQMHistoSizes: changed ( 10224.0,... ): 2132.034 KiB Muons/RecoDisplacedMuonV
- Checked 202 log files, 165 edm output root files, 48 DQM output files
- TriggerResults: no differences found
please test
+1
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-111b89/38456/summary.html
COMMIT: 2785e6ec7a2c63601f15c445f68d05dfdf53dbc3
CMSSW: CMSSW_14_1_X_2024-03-26-2300/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/44353/38456/install.sh to create a dev area with all the needed externals and cmssw changes.
Comparison Summary
Summary:
- You potentially removed 105 lines from the logs
- Reco comparison results: 46 differences found in the comparisons
- DQMHistoTests: Total files compared: 48
- DQMHistoTests: Total histograms compared: 3297857
- DQMHistoTests: Total failures: 6
- DQMHistoTests: Total nulls: 0
- DQMHistoTests: Total successes: 3297831
- DQMHistoTests: Total skipped: 20
- DQMHistoTests: Total Missing objects: 0
- DQMHistoSizes: Histogram memory added: 29848.476 KiB( 47 files compared)
- DQMHistoSizes: changed ( 10224.0,... ): 2132.034 KiB Muons/RecoDisplacedMuonV
- Checked 202 log files, 165 edm output root files, 48 DQM output files
- TriggerResults: no differences found
+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 now be reviewed by the release team before it's merged. @antoniovilela, @sextonkennedy, @rappoccio (and backports should be raised in the release meeting by the corresponding L2)
+1