-
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: ITk seeding example #1186
feat: ITk seeding example #1186
Conversation
Not sure we want to add the "old-style" example in the repository, also because it includes a specific ITk implementation of it. Can we try to have the example implemented using the python-binding framework? |
Codecov Report
@@ Coverage Diff @@
## main #1186 +/- ##
=======================================
Coverage 47.81% 47.81%
=======================================
Files 360 360
Lines 18591 18591
Branches 8769 8769
=======================================
Hits 8890 8890
Misses 3649 3649
Partials 6052 6052
📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more |
We should discuss if an example containing all the configuration to run the ITk seeding should be included into ACTS or just an example showing how to configure the new seeding features. In any case, this example still does not contain the full configuration to produce a 1 to 1 comparison to ITk pixel seeds. |
…elho/acts into ITk-seeding-example
…elho/acts into ITk-seeding-example
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.
Ok let's go. Would you be able to follow up this PR with another one which adds this example to the python level automatic tests (so we can ensure this keeps running)?
ok! I will work on that @paulgessinger |
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