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
trackingNtuple updates from mkFit/LST developments #34256
trackingNtuple updates from mkFit/LST developments #34256
Conversation
…t and the electrons are asked to be saved
… simhits associated to the rechit based on minimal significance of the local position distance
…iterative tracking
…k building interation), read all available cases by default: detached and lowPt Quad (pixel seeds) ; highPt,lowPt, and detached Triplet (pixel seeds); pixelPair; mixedTriplet, pixelLess and tobTec
…he fitted tracks from each iteration are collected/combined via an instance of TrackSimpleMerger, process.mergedStepTracks); useful for individual analysis of iterations
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-34256/23539
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
b43ceac
to
be2d604
Compare
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-34256/23540
|
A new Pull Request was created by @slava77 (Slava Krutelyov) for master. It involves the following packages: CommonTools/RecoAlgos @perrotta, @andrius-k, @kmaeshima, @ErnestaP, @ahmad3213, @cmsbuild, @jfernan2, @slava77, @jpata, @rvenditti can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-1d441c/16249/summary.html Comparison SummarySummary:
|
+1 |
@mtosi @vmariani @mmusich I see that DQM have signed already, but since I did not discuss the details in this PR explicitly with TRK POG, I wanted to get an OK for the updates. (this was reviewed/discussed with former TRK POG conveners (@makortel and @cerati ) as a part of developments/needs for mkFit; just to give more context) |
+1, thanks |
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. @silviodonato, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
these updates accumulated over a couple of years in the mkFit and line-segment tracking (LST) developments.
The default trackingNtuple logic should be the same (aside for a few added branches).
Summary of changes relative to the upstream in TrackingNtuple
includeOOT
keepEleSimHits
saveSimHitsP3
simHitBySignificance
addSeedCurvCov
in a row-major vector of size 15clusterMasks
customiseTrackingNtupleHLT
to be able to run on HLT trackingAlso, add
TrackSimpleMerger
which can concatenate a set of reco::Track collections.