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: Make Clusterization.hpp aware of grid dimensionality #1634
feat: Make Clusterization.hpp aware of grid dimensionality #1634
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1634 +/- ##
==========================================
+ Coverage 49.65% 49.81% +0.15%
==========================================
Files 404 406 +2
Lines 22435 22547 +112
Branches 10258 10293 +35
==========================================
+ Hits 11141 11232 +91
- Misses 4132 4136 +4
- Partials 7162 7179 +17
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
📊 Physics performance monitoring for 72b1fd2Full report VertexingSeedingCKFAmbiguity resolutionTruth tracking (Kalman Filter)Truth tracking (GSF) |
dd1e405
to
75bfc94
Compare
Running an integration test on the athena side before un-WIP-ing |
9c7c1d6
to
e22400c
Compare
Ready for review. Perhaps @asalzburger or @paulgessinger could review? Or @noemina ? |
ac3360c
to
5c88382
Compare
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.
Nothing to mention, all looks good to me.
This PR makes Clusterization.hpp aware of the grid dimensionality. Since the problem is much simpler on a 1-D grid, this enables a few optimizations. Right now, only support for 2D and 1D grid is implemented.
To aid in testing, unit tests for both cases have also been added.
Note, the changes are supposed to be backwards compatible. Previously it was assumed that the grid was 2-dimensional, thus the GridDim template parameter defaults to 2 where relevant.