-
Notifications
You must be signed in to change notification settings - Fork 178
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: move initialisation of vector in SeedFilter #2135
refactor: move initialisation of vector in SeedFilter #2135
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2135 +/- ##
=======================================
Coverage ? 49.43%
=======================================
Files ? 434
Lines ? 25003
Branches ? 11523
=======================================
Hits ? 12359
Misses ? 4490
Partials ? 8154
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
📊 Physics performance monitoring for dd73884Summary VertexingSeedingCKFAmbiguity resolutionTruth tracking (Kalman Filter)Truth tracking (GSF) |
did you reach a conclusion on this one @LuisFelipeCoelho @CarloVarni ? |
@LuisFelipeCoelho @andiwand I'm ok with this PR, but I cannot resolve the discussion... looks like github is not allowing the one opening the discussion to close it ... |
This PR moves the initialisation of
compatibleSeedR
in seedFilter to reduce memory allocation