Skip to content

Draft implementation of a CPU vs. GPU workflow and infrastructure for Phase 2 HLT tracking#235

Closed
VourMa wants to merge 1 commit intonewTRKAsPhase2HLTDefault_bckfrom
GPUValidationForPhase2HLTTRK_draft
Closed

Draft implementation of a CPU vs. GPU workflow and infrastructure for Phase 2 HLT tracking#235
VourMa wants to merge 1 commit intonewTRKAsPhase2HLTDefault_bckfrom
GPUValidationForPhase2HLTTRK_draft

Conversation

@VourMa
Copy link
Collaborator

@VourMa VourMa commented Feb 4, 2026

This is draft PR, aiming to be submitted only after the new tracking baseline is merged in cms-sw (hence the target branch is newTRKAsPhase2HLTDefault. It is opened just so that I can gather feedback on the new implementation of the CPU vs. GPU validation for Phase 2 HLT tracking that this time is generalized to look at built tracks as well as pixel tracks, and can be extended to look at LST tracks (tracks from the LST seeds used in the new tracking baseline) as well, if we feel this is useful.

More in general, the PR provides a direction on how to implement CPU vs. GPU validations for Phase 2 HLT for different objects: paths like DQM_TRKHeterogeneousValidation can be created for other objects and then added to the "heterogeneous validation" menu, HLT_HeterogeneousValid.

This partly supersedes the HLT part of #215, where some updates to the TrackToTrackComparisonHists producer are needed (especially because this producer has been updated in a recent cms-sw PR).

Example result from the newly introduced workflow:
image

@VourMa VourMa force-pushed the newTRKAsPhase2HLTDefault branch from 5d07346 to 1cb2ecc Compare February 5, 2026 08:58
Copy link

@slava77 slava77 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure the path files should clone/define new producer modules or worse yet redefine already existing sequences; perhaps I'm missing some design constraints

Comment on lines +18 to +19
HLTTrackingSequenceCommon_HeterogeneousValidation = cms.Sequence(
HLTItLocalRecoSequence
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have doubts this file should redefine the reference sequence to be validated. Normally I'd expect the reference should be taken in full from the standard path; perhaps HLTMkFitInputSequence

If the idea is to avoid mis-configurations in the SerialSync mirror, perhaps it's better to define these mirror modules and sequences in the places where the reference is defined (check with Marco if the config parsing/validation tools may introduce constraints)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Another good point, I will switch to this approach. I avoided it because previously it gave issues with the combinations of procModifiers but it should be OK if run only on the default.

@VourMa VourMa force-pushed the newTRKAsPhase2HLTDefault branch 2 times, most recently from 81b2882 to e72f9c1 Compare February 9, 2026 15:53
@VourMa VourMa changed the base branch from newTRKAsPhase2HLTDefault to newTRKAsPhase2HLTDefault_bck February 11, 2026 16:22
@slava77 slava77 mentioned this pull request Feb 11, 2026
12 tasks
@VourMa
Copy link
Collaborator Author

VourMa commented Feb 20, 2026

Superseded by #240

@VourMa VourMa closed this Feb 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants