RecoHGCal/TICL: avoid NaN energies in PF and superclustering

[felicepantaleo]

• PFTICLProducer: guard the EM/total energy fraction when a candidate has no tracksters (track-only/global-muon) -> no NaN ECAL energy on the PFCandidate.
• EGammaSuperclusterProducer: initialise the running maxima with numeric_limits::lowest() instead of ::min(), which corrupted etaWidth/phiWidth regression features.
• TracksterP4FromEnergySumPlugin: guard the barycentre/energy division (> 0, not >= 0).

[cmsbuild]

cms-bot internal usage

[felicepantaleo]

@cmsbuild please test

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-51341/49973

• There are other open Pull requests which might conflict with changes you have proposed:
  • File RecoHGCal/TICL/plugins/EGammaSuperclusterProducer.cc modified in PR(s): Add directories for TICL-EGM code development #48319
  • File RecoHGCal/TICL/plugins/PFTICLProducer.cc modified in PR(s): Add directories for TICL-EGM code development #48319

[cmsbuild]

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

It involves the following packages:

• RecoHGCal/TICL (reconstruction)

@Moanwar, @jfernan2, @mandrenguyen, @srimanob can you please review it and eventually sign? Thanks.
@apsallid, @denizsun, @hatakeyamak, @lecriste, @rovere, @salimcerci, @sameasy, @sobhatta this is something you requested to watch as well.
@ftenchini, @mandrenguyen, @sextonkennedy you are the release manager for this.

cms-bot commands are listed here

[felicepantaleo]

type bug fix

[felicepantaleo]

type bugfix

[cmsbuild]

+1

Size: This PR adds an extra 28KB to repository
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-769af1/54332/summary.html
COMMIT: ad279ea
CMSSW: CMSSW_20_1_X_2026-06-27-1100/el9_amd64_gcc13
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/51341/54332/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: 230 differences found in the comparisons
• DQMHistoTests: Total files compared: 45
• DQMHistoTests: Total histograms compared: 3414477
• DQMHistoTests: Total failures: 430
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3414029
• DQMHistoTests: Total skipped: 18
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 44 files compared)
• Checked 195 log files, 163 edm output root files, 45 DQM output files
• TriggerResults: no differences found