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
implement a tracking particle associator with TrackCandidate and add trackCandidate branches in trackingNtuple #37829
implement a tracking particle associator with TrackCandidate and add trackCandidate branches in trackingNtuple #37829
Conversation
…update the TrackToTrackingParticleAssociator interface for TrackCandidate associations to be done wrt RefVector of TrackingParticles
…ntation templates to support different reference track types: now supporting both reco:Track and TrackCandidate
…rs case to include track MVAs
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37829/29745 ERROR: Build errors found during clang-tidy run.
|
@smuzaffar @makortel |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37829/29748
|
A new Pull Request was created by @slava77 (Slava Krutelyov) for master. It involves the following packages:
@civanch, @emanueleusai, @mdhildreth, @cmsbuild, @jfernan2, @ahmad3213, @pmandrik, @micsucmed, @rvenditti can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild please test |
@slava77 , |
OK. In the end I have probably just missed the error when I ran code-checks locally. |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-baef6c/24485/summary.html Comparison SummarySummary:
|
as expected there are no relevant differences (apart for a few false-positives from the warning messages in PU workflows). |
+1 |
+1 |
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. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
SimToRecoCollection*
andRecoToSimCollection*
and removed its partial duplicate definition in TrackToTrackingParticleAssociator.h to keep everything just in TrackAssociation.h. The fully defined types should be the same, so no update should be necessary in theclasses*
files for the dictionaries. There are no use cases at this point for placingSimToRecoCollectionTCandidate
andRecoToSimCollectionTCandidate
inedm::Event
; so, no dictionary is added for this.TrackToTrackingParticleAssociator::associateRecoToSim
arguments/interface fromHandle<TrackingParticleCollection
toRefVector<TrackingParticleCollection
to be more in common with existing use cases forreco::Tracks
QuickTrackAssociatorByHitsImpl
so thatreco::Track
andTrackCandidate
have maximal overlap in implementation and minimize copy-pasted codetcand_*
branches to provide access toTrackCandidate
details (optional, enabled for extendedContent custom option). This should allow more straightforward debugging of track pattern recognition, in particular in mkFit. This data can also be used to train track candidate classification and allow removing candidates from the following final fit step (and save computation)TransientTrackingRecHitBuilder
and use tracking hits directly. This is in a sense a somewhat late follow-up to TTRHs removed from Tracker code for good. #3344No changes are expected in the standard workflows.
If that is confirmed, there may be a backport to an older release (12_2_X where the TRK-Run3Winter22 are made, in case it's not practical to run on those inputs in 12_4_X)