-
Notifications
You must be signed in to change notification settings - Fork 157
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
refactor: replace track params with trajectories in examples #1670
refactor: replace track params with trajectories in examples #1670
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1670 +/- ##
=======================================
Coverage 48.54% 48.54%
=======================================
Files 383 383
Lines 20976 20976
Branches 9641 9641
=======================================
Hits 10182 10182
Misses 4119 4119
Partials 6675 6675 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
📊 Physics performance monitoring for 116e29bFull report VertexingCKFAmbiguity resolutionTruth tracking |
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.
Very nice, I only would like this common utility function to remove at least a bit code duplication.
Examples/Algorithms/TruthTracking/ActsExamples/TruthTracking/TrackSelector.cpp
Outdated
Show resolved
Hide resolved
Examples/Algorithms/TruthTracking/ActsExamples/TruthTracking/TrackSelector.cpp
Outdated
Show resolved
Hide resolved
Examples/Algorithms/TruthTracking/ActsExamples/TruthTracking/TrackSelector.hpp
Outdated
Show resolved
Hide resolved
Examples/Algorithms/Vertexing/src/IterativeVertexFinderAlgorithm.cpp
Outdated
Show resolved
Hide resolved
looks like this is passing now @benjaminhuth cc @paulgessinger if you want to check this out |
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.
looks good i think
…oject#1670) somehow we ended up passing through track parameters and trajectory tips through our algorithm chain which can be replaced by trajectories the only caveat is particle smearing which will produce track parameters which should be passed down the chain. so track selector and vertexing will still except track parameters as an input
somehow we ended up passing through track parameters and trajectory tips through our algorithm chain which can be replaced by trajectories
the only caveat is particle smearing which will produce track parameters which should be passed down the chain. so track selector and vertexing will still except track parameters as an input