[14_2_X] Update ecalMultiFitUncalibRecHit_cfi.py parameters for CC

[jking79]

BackPort of PR #46763
This PR does not affect the default reconstruction. The changes made by this PR are only activated by a special modifier. This PR is needed for a reprocessing discussed with PPD.
Changed ourOfTimeThresholdGain61pEB and ourOfTimeThresholdGain61mEB parameters for CC reconstruction from 3.0 to 12.0 on lines 15 and 16 of RecoLocalCalo/EcalRecProducers/python/ecalMultiFitUncalibRecHit_cfi.py
PR description:
This change address discrepancy seen in JERSF for TeV jets.
PR validation:
This PR makes an adjustment to config parameters and does not touch the code.
Details are documented in: SupplementalMaterialPR46763.pdf

[cmsbuild]

A new Pull Request was created by @jking79 for CMSSW_14_2_X.

It involves the following packages:

• RecoLocalCalo/EcalRecProducers (reconstruction)

@cmsbuild, @jfernan2, @mandrenguyen can you please review it and eventually sign? Thanks.
@ReyerBand, @apsallid, @argiro, @denizsun, @missirol, @rchatter, @salimcerci, @thomreis, @wang0jin, @youyingli 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

• Backported from Update ecalMultiFitUncalibRecHit_cfi.py parameters for CC #46763

[cmsbuild]

cms-bot internal usage

[jfernan2]

please test

[jfernan2]

assign alca

[cmsbuild]

New categories assigned: alca

@atpathak,@consuegs,@perrotta you have been requested to review this Pull request/Issue and eventually sign? Thanks

[jfernan2]

type ecal

[jfernan2]

backport of #46763

[perrotta]

This PR does not affect the default reconstruction. The changes made by this PR are only activated by a special modifier. This PR is needed for a reprocessing discussed with PPD.

@jking79 I don't see the modifier you are talking about. Could you please point it out?

In which release is that re-processing expected to be run?

[jking79]

@perrotta PR 46914 and PR 46915 where made at the request of @mandrenguyen to extend the changes made in PR 46904 for CMSSW_14_0_X to CMSSW_14_1_X and CMSSW 14_2_X. The reprocessing will be done in CMSSW_14_0_X. This PR makes parameter changes for the ECAL CC time reconstruction. The special modifier mentioned is used to switch the ECAL time reconstruction to the CC time reconstruction. This modifier is not included in this PR.

[cmsbuild]

+1

Size: This PR adds an extra 16KB to repository
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-1870af/43391/summary.html
COMMIT: 322f85e
CMSSW: CMSSW_14_2_X_2024-12-11-1100/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/46915/43391/install.sh to create a dev area with all the needed externals and cmssw changes.

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:

• @enibigir CMSSW_14_2_X:Update HSCParticle object to be compatible with MiniAOD #46900

You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-1870af/43391/git-recent-commits.json
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-1870af/43391/git-merge-result

Comparison Summary

Summary:

• No significant changes to the logs found
• Reco comparison results: 8 differences found in the comparisons
• DQMHistoTests: Total files compared: 46
• DQMHistoTests: Total histograms compared: 3483722
• DQMHistoTests: Total failures: 478
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3483224
• DQMHistoTests: Total skipped: 20
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 45 files compared)
• Checked 202 log files, 172 edm output root files, 46 DQM output files
• TriggerResults: no differences found

[thomreis]

Hi @perrotta the modifier is ecal_cctiming.

[perrotta]

Hi @perrotta the modifier is ecal_cctiming.

Thank you @thomreis
As it was only mentioned in the description of these backport PRs I suspected that it was some special one instead, meant to a dedicated reprocessing and not yet considered here. Therefore, in the end this PR is functionally identical to the one already merged in master, and the new parameters will be set when the cctiming algo will be activated.

[perrotta]

+alca

• I would suggest either @jking79 or @cms-sw/orp-l2 to modify the title of the PR into something more easily understandable, for example "Update ecalMultiFitUncalibRecHit_cfi.py parameters for CC - CMSSW_14_2_X"

[jfernan2]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_14_2_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_15_0_X is complete. This pull request will now be reviewed by the release team before it's merged. @sextonkennedy, @rappoccio, @antoniovilela, @mandrenguyen (and backports should be raised in the release meeting by the corresponding L2)

[mandrenguyen]

backport of PR #46763

[mandrenguyen]

backport of #46763

[antoniovilela]

+1