deprecate integer types for L1T and HLT prescales in HLTConfigData and its clients [12_5_X]

[missirol]

backport of #39220

PR description:

A full description of the scope and implementation of this PR can be found in #39220.

Summary:

• support only double and FractionalPrescale as valid types to access the values of L1T and HLT prescales in HLTConfigData and all of its clients (e.g. HLTConfigProvider, HLTPrescaleProvider, PATTriggerProducer, etc);
• client code in CMSSW is updated accordingly; user code outside CMSSW that uses these methods will run into a compile-time error, in order to force users to specify one of the supported types (avoiding silent conversion of non-integer values into integers in user code);
• a bi-product of this PR is a bugfix to MINIAOD, which has so far mistakenly saved L1T prescales as integers (even though in Run 3 L1T prescales can actually have non-integer values); this requires a change in DataFormats/PatCandidates.

Verbatim backport.

PR validation:

Relies on the validation done for the original PR.

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:

#39220

Bugfix for MINIAOD.

[cmsbuild]

A new Pull Request was created by @missirol (Marino Missiroli) for CMSSW_12_5_X.

It involves the following packages:

• Alignment/OfflineValidation (alca)
• Calibration/HcalCalibAlgos (alca)
• Calibration/IsolatedParticles (alca)
• CommonTools/TriggerUtils (hlt, l1)
• DPGAnalysis/Skims (pdmv)
• DQM/Physics (dqm)
• DQMOffline/Trigger (dqm)
• DataFormats/PatCandidates (reconstruction)
• HLTrigger/HLTanalyzers (hlt)
• HLTrigger/HLTcore (hlt)
• PhysicsTools/Heppy (analysis)
• PhysicsTools/PatAlgos (reconstruction)
• PhysicsTools/TagAndProbe (analysis)
• Validation/RecoVertex (dqm)

@syuvivida, @bbilin, @Martin-Grunewald, @rekovic, @ChrisMisan, @cecilecaillol, @yuanchao, @ahmad3213, @cmsbuild, @missirol, @mandrenguyen, @pmandrik, @epalencia, @emanueleusai, @jfernan2, @kskovpen, @micsucmed, @francescobrivio, @malbouis, @clacaputo, @saumyaphor4252, @sunilUIET, @rvenditti, @tvami can you please review it and eventually sign? Thanks.
@rappoccio, @gouskos, @VourMa, @felicepantaleo, @hatakeyamak, @emilbols, @Martin-Grunewald, @bsunanda, @tlampen, @mbluj, @ahinzmann, @demuller, @seemasharmafnal, @venturia, @mmarionncern, @silviodonato, @JanFSchulte, @jhgoh, @jdolen, @HuguesBrun, @azotz, @cericeci, @fwyzard, @trocino, @rociovilar, @GiacomoSguazzoni, @rovere, @VinInn, @jdamgov, @nhanvtran, @gkasieczka, @tocheng, @schoef, @mmusich, @mtosi, @dgulhan, @AlexDeMoor, @adewit, @AnnikaStein, @Fedespring, @JyothsnaKomaragiri, @gpetruc, @mariadalfonso, @andrzejnovak 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

• Backported from deprecate integer types for L1T and HLT prescales in HLTConfigData and its clients #39220

[missirol]

type bugfix

#39220 (comment) : a backport to 12_5_X was asked because of the bugfix to MINIAOD.

[missirol]

please test

[cmsbuild]

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-e667f4/27652/summary.html
COMMIT: 6c0922f
CMSSW: CMSSW_12_5_X_2022-09-19-1100/el8_amd64_gcc10
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/39446/27652/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

• No significant changes to the logs found
• Reco comparison results: 4 differences found in the comparisons
• DQMHistoTests: Total files compared: 51
• DQMHistoTests: Total histograms compared: 3699454
• DQMHistoTests: Total failures: 8
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3699424
• DQMHistoTests: Total skipped: 22
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 50 files compared)
• Checked 212 log files, 49 edm output root files, 51 DQM output files
• TriggerResults: no differences found

[missirol]

+hlt

To be clear, this backport is not strictly needed for HLT. The main beneficiary is MINIAOD.

[tvami]

+alca

[cecilecaillol]

+l1

[mandrenguyen]

+1

[missirol]

@cms-sw/pdmv-l2 @cms-sw/dqm-l2

Do you have any comments on this backport?

[tvami]

urgent

• needed for today's release

[kskovpen]

+pdmv

[emanueleusai]

+1

[rappoccio]

+1