-
Notifications
You must be signed in to change notification settings - Fork 176
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
fix: Update RootTrajectorySummaryWriter.cpp - writing the correct vector to branch (outlierChi2) #2488
fix: Update RootTrajectorySummaryWriter.cpp - writing the correct vector to branch (outlierChi2) #2488
Conversation
Note: once we're done, I will merge this one manually so @altsybee shows up in the merge commit correctly. |
# Conflicts: # CI/physmon/reference/performance_ckf_seeded.root # CI/physmon/reference/performance_ckf_truth_smeared.root # Examples/Python/tests/root_file_hashes.txt
Codecov Report
@@ Coverage Diff @@
## main #2488 +/- ##
=======================================
Coverage 49.77% 49.77%
=======================================
Files 466 466
Lines 26310 26310
Branches 12093 12093
=======================================
Hits 13095 13095
Misses 4616 4616
Partials 8599 8599 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
# Conflicts: # CI/physmon/reference/tracksummary_ckf_orthogonal_hist.root # CI/physmon/reference/tracksummary_ckf_seeded_hist.root # CI/physmon/reference/tracksummary_ckf_truth_estimated_hist.root # CI/physmon/reference/tracksummary_ckf_truth_smeared_hist.root # Examples/Python/tests/root_file_hashes.txt
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.
lets go!
Merged this one manually so @altsybee is mentioned. |
Failure is the metadata failure. |
Fix: writing the correct vector to branch: outlierChi2 instead of measurementChi2.
Reopened #2363 so I can update the references.