-
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
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:
|
+1
|
+Upgrade |
+pdmv |
+1 |
@cms-sw/dqm-l2 w.r.t. the latest sign-off I only removed a couple of comments. Could you have a look? Thanks ( : |
+dqm |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @mandrenguyen, @antoniovilela, @sextonkennedy, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
fix replacing of split pixel clusters in DQM online clients after #46694
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