Miscellaneous fixes for HLT online DQM

[mmusich]

PR description:

This PR bundles together a couple of fixes for the HLT online DQM:

• I noticed looking at the Pixel residuals @HLT plots in recent runs that they were empty. Looking at the dqm-square log for that run, I also saw a bunch of errors of the type:

%MSG-w TrackProducerBase:  TrackRefitter:hltrefittedForPixelDQM  03-Oct-2024 22:56:25 CEST Run: 386554 Event: 2607599
BeamSpot is not valid
%MSG
%MSG-e TrackRefitter:  TrackRefitter:hltrefittedForPixelDQM  03-Oct-2024 22:56:25 CEST Run: 386554 Event: 2607599
BeamSpot is (0,0,0), it is probably because is not valid in the event
%MSG

I believe the issue is that the current hltrefittedForPixelDQM instance clones the one used for offline (refittedForPixelDQM) which ultimately clones the template with the offline parameters:

cmssw/RecoTracker/TrackProducer/python/TrackRefitter_cfi.py

Line 5 in 180e6a5

beamSpot = cms.InputTag("offlineBeamSpot"),

• secondly, also looking at the HLT quick collection many of the plots are empty, I believe due the usage of deprecated trigger paths in input

The two issues are addressed at 3bf61e3 and 17e3bf7 respectively.

PR validation:

None, will need to be tested via DQM replay

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, might backport to 14.1.x and 14.0.x for 2024 data-taking purposes.

[cmsbuild]

cms-bot internal usage

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-46245/42070

[cmsbuild]

A new Pull Request was created by @mmusich for master.

It involves the following packages:

• DQM/HLTEvF (hlt, dqm)
• DQMOffline/Trigger (dqm)

@Martin-Grunewald, @antoniovagnerini, @cmsbuild, @mmusich, @nothingface0, @rvenditti, @syuvivida, @tjavaid can you please review it and eventually sign? Thanks.
@Fedespring, @HuguesBrun, @cericeci, @jhgoh, @missirol, @mtosi, @rociovilar, @silviodonato, @trocino this is something you requested to watch as well.
@antoniovilela, @mandrenguyen, @rappoccio, @sextonkennedy you are the release manager for this.

cms-bot commands are listed here

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-46245/42072

[cmsbuild]

Pull request #46245 was updated. @Martin-Grunewald, @antoniovagnerini, @cmsbuild, @mmusich, @nothingface0, @rvenditti, @syuvivida, @tjavaid can you please check and sign again.

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-46245/42076

[cmsbuild]

Pull request #46245 was updated. @Martin-Grunewald, @antoniovagnerini, @cmsbuild, @mmusich, @nothingface0, @rvenditti, @syuvivida, @tjavaid can you please check and sign again.

[mmusich]

@cmsbuild, please test

[mmusich]

playback kindly provided by @nothingface0 looks good.

Pixel residuals:

Object Monitor:

[cmsbuild]

+1

Size: This PR adds an extra 28KB to repository
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-73dcff/41962/summary.html
COMMIT: 3c5e8fe
CMSSW: CMSSW_14_2_X_2024-10-04-1100/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/46245/41962/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

• You potentially added 2 lines to the logs
• Reco comparison results: 48 differences found in the comparisons
• DQMHistoTests: Total files compared: 44
• DQMHistoTests: Total histograms compared: 3331336
• DQMHistoTests: Total failures: 6060
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3325256
• DQMHistoTests: Total skipped: 20
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 43 files compared)
• Checked 193 log files, 163 edm output root files, 44 DQM output files
• TriggerResults: no differences found

[mmusich]

+hlt

• DQM offline comparisons at Miscellaneous fixes for HLT online DQM  #46245 (comment) look good
• DQM online playback looks good (see Miscellaneous fixes for HLT online DQM  #46245 (comment))

[antoniovagnerini]

+1

• spurious differences seen from unstable WF 12834.7 (open issue spurious differences in outputs of Run-3 wfs *.7 #39803 )

[cmsbuild]

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. @rappoccio, @sextonkennedy, @antoniovilela, @mandrenguyen (and backports should be raised in the release meeting by the corresponding L2)

[mandrenguyen]

+1