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
test: Unit test geometric digitization hit merging #2213
test: Unit test geometric digitization hit merging #2213
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2213 +/- ##
=======================================
Coverage 49.35% 49.35%
=======================================
Files 445 445
Lines 25263 25263
Branches 11650 11650
=======================================
Hits 12468 12468
Misses 4511 4511
Partials 8284 8284 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Tests/UnitTests/Examples/Algorithms/Digitization/ModuleClustersTests.cpp
Outdated
Show resolved
Hide resolved
…sTests.cpp Co-authored-by: Alexander J. Pfleger <70842573+AJPfleger@users.noreply.github.com>
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! Feel free to ignore the one comment!
Tests/UnitTests/Examples/Algorithms/Digitization/ModuleClustersTests.cpp
Outdated
Show resolved
Hide resolved
Tests/UnitTests/Examples/Algorithms/Digitization/ModuleClustersTests.cpp
Outdated
Show resolved
Hide resolved
Tests/UnitTests/Examples/Algorithms/Digitization/ModuleClustersTests.cpp
Outdated
Show resolved
Hide resolved
📊 Physics performance monitoring for bfff96aSummary VertexingSeedingCKFAmbiguity resolutionTruth tracking (Kalman Filter)Truth tracking (GSF) |
This test lives in examples because the cluster class is an examples one, right? Also concerning the label: I think the pattern would have to be updated for this to be recognized as an Examples change |
Not sure if the bot should modify manually added or removed labels. But I guess there are pros and cons about this |
This adds a unit test to the geometric digitization. For now I just added a quick check if that hit merging does something if you provide the same hit twice.