-
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
feat: Update RootVertexPerformanceWriter to work with track finding #1460
feat: Update RootVertexPerformanceWriter to work with track finding #1460
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1460 +/- ##
=======================================
Coverage 48.60% 48.60%
=======================================
Files 380 380
Lines 20595 20595
Branches 9433 9433
=======================================
Hits 10010 10010
Misses 4097 4097
Partials 6488 6488 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Co-authored-by: Andreas Stefl <stefl.andreas@gmail.com>
Ok I fixed one source of inconsistency resulting in output changes, and then checked that the remaining output changes we see here only come from the newly added branches. I'll let the CI run and update the hashes, and then I think this is good to go. |
Previously, the
RootVertexPerformanceWriter
assumed a 1:1 correspondence between truth particles and reconstructed tracks. This PR implements hit based truth matching to enable this to run.