RecoHGCal/TICL: linking and inference fixes#51342
Conversation
felicepantaleo
commented
Jun 28, 2026
- TracksterLinkingbySkeletons: use abs(deltaPhi) in the phi window (was signed).
- LinkingAlgoByDirectionGeometric: wrap the phi difference (reco::deltaPhi) in the cut.
- GNNInterpretationAlgo: make the search window and squared distance cut consistent.
- TracksterInferenceByDNN: skip barrel clusters in the tensor fill (consistent with the selection loop); fix the miniBatchSize default (64).
- TrackstersProducer: run barrel inference on the produced tracksters, not the empty initialResult.
- TracksterLinkingbySkeletons: use abs(deltaPhi) in the phi window (was signed). - LinkingAlgoByDirectionGeometric: wrap the phi difference (reco::deltaPhi) in the cut. - GNNInterpretationAlgo: make the search window and squared distance cut consistent. - TracksterInferenceByDNN: skip barrel clusters in the tensor fill (consistent with the selection loop); fix the miniBatchSize default (64). - TrackstersProducer: run barrel inference on the produced tracksters, not the empty initialResult.
|
@cmsbuild please test |
|
cms-bot internal usage |
|
@cmsbuild please test |
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-51342/49974 |
|
A new Pull Request was created by @felicepantaleo for master. It involves the following packages:
@Moanwar, @jfernan2, @mandrenguyen, @srimanob can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
|
type bugfix |
|
+1 Size: This PR adds an extra 44KB to repository Comparison SummarySummary:
|
|
type ngt |