-
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!: changed Grid interface of neighborHoodIndices #1052
feat!: changed Grid interface of neighborHoodIndices #1052
Conversation
…d upper neighbors
Codecov Report
@@ Coverage Diff @@
## main #1052 +/- ##
==========================================
- Coverage 48.57% 48.57% -0.01%
==========================================
Files 338 338
Lines 17343 17369 +26
Branches 8200 8216 +16
==========================================
+ Hits 8425 8437 +12
+ Misses 3180 3179 -1
- Partials 5738 5753 +15
Continue to review full report at Codecov.
|
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. If the tests pass, let's merge it.
This PR adds the `ITkSeedingExample.cpp` to run the ITk seeding. It shows how to configure the non-equidistant binning in z (#1005), the seed confirmation cuts (#1084), the radial range for middle SP cut (#1084) and the vector containing the map of z neighbours (#1052 and #1038). It contains all the parameters to run the seeding for ITk **pixel** space points (I intend to extend this to ITk strip SPs soon). @noemina @paulgessinger
Changed
Grid
interface ofneighborHoodIndices
with differing lower and upper neighbors. This now allows to specify the number of neighbors desired per axis, as opposed to having one number of neighbors for all axes. It also inverts the way neighbors are interpreted:BREAKING CHANGE: The
Grid
interface changes: f you want 1 neighbor on the left and two on the right, you now have to write{-1,2}
instead of{1,2}
as before. This is necessary as this also allows to give "negative" neighbors, i.e. not starting from the current bin, but e.g. only 2 bins left of the current bin:{-2,-1}
.