-
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: Add variable axis type for SpacePointGrid - Follow-up #1005
feat: Add variable axis type for SpacePointGrid - Follow-up #1005
Conversation
In other to check if the variable binning significantly slows down the execution, I generated 1000 ttbar events using pythia8 and FatrasTGeo using the ATLAS response and ran the seeding example 10 times for the equidistant and variable binning with the same 100x100 grid. The average time spent per event using the equidistant binning was 0.045295s and using the variable binning was 0.045979s. The same results were achieved in almost the same time and we believe that it may not be worth adding more complications.
…ho/acts into variable-axistype
Codecov Report
@@ Coverage Diff @@
## main #1005 +/- ##
=======================================
Coverage 48.60% 48.60%
=======================================
Files 337 337
Lines 17330 17329 -1
Branches 8191 8191
=======================================
Hits 8423 8423
+ Misses 3166 3165 -1
Partials 5741 5741
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.
👍
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
I lost access to the PR #976 so I am creating this follow-up PR in order to fix some minor comments.