-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allowing Cluster Splitting with HLT Vertices #46694
base: master
Are you sure you want to change the base?
Conversation
type tracking |
cms-bot internal usage |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-46694/42636
|
A new Pull Request was created by @AdrianoDee for master. It involves the following packages:
@AdrianoDee, @Moanwar, @antoniovagnerini, @antoniovilela, @cmsbuild, @davidlange6, @DickyChant, @fabiocos, @jfernan2, @mandrenguyen, @miquork, @rappoccio, @rseidita, @srimanob, @subirsarkar can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
test parameters:
|
please test |
def setup_(self, step, stepName, stepDict, k, properties): | ||
stepDict[stepName][k] = merge([{'--procModifiers': 'hltClusterSplitting'}, stepDict[step][k]]) | ||
# if step in ["Digi","HLTOnly","DigiTrigger"]: | ||
# stepDict[stepName][k] = merge([{'--customise': 'RecoLocalTracker/SubCollectionProducers/customizeSplittingFromHLT.customiseSplittingFromHLT'}, stepDict[stepName][k]]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Forgot to remove these comments.
|
||
hltClusterSplitting.toModify(siPixelClusters, | ||
vertices = cms.InputTag("hltPixelVertices") | ||
# hltPoint = cms.InputTag("hltTrimmedPixelVertices"), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also these, will do a cleanup after the tests.
+1 Size: This PR adds an extra 60KB to repository Comparison SummarySummary:
|
PR description:
This PR proposes the introduction of a new process modifier
hltClusterSplitting
that:hltPixelVertices
inFEVTDEBUGHLTEventContent
;hltTrimmedPixelVertices
that are just 1-2 for Run3 HLT;ak4CaloJetsForTrkPreSplitting
and the cluster splitter;InitialStepPreSplitting
iteration from iterative tracking.Running on 14_2_0_pre2 RelVals with Run3 PU:
the tracking performance seems untouched.
I don't know how much this would be useful for Run3, but it may be interesting to check it for Phase2. For Run3 I've measured (roughly) a speedup of 1.5-2% on a TTbar Run3 PU 2025 conditions sample.
I took the chance also to add a specific wf with offset
.19003
(to be aligned to the other cluster splitting related wfs) and aProdLike
version with.1900321
.PR validation:
runTheMatrix.py -w upgrade -l 17434.19003,17434.1900321