-
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: More robust navigation with bent tracks #1769
feat: More robust navigation with bent tracks #1769
Conversation
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.
I guess it's "bent" tracks.
Codecov Report
@@ Coverage Diff @@
## main #1769 +/- ##
=======================================
Coverage 49.83% 49.83%
=======================================
Files 406 406
Lines 22532 22532
Branches 10280 10280
=======================================
Hits 11228 11228
Misses 4131 4131
Partials 7173 7173
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Co-authored-by: Paul Gessinger <hello@paulgessinger.com>
Co-authored-by: Alexander J. Pfleger <70842573+AJPfleger@users.noreply.github.com>
📊 Physics performance monitoring for 8974c57Full report VertexingSeedingCKFAmbiguity resolutionTruth tracking (Kalman Filter)Truth tracking (GSF) |
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.
Few more typos, otherwise good to go!
Co-authored-by: Paul Gessinger <hello@paulgessinger.com>
This PR adds an option to the
Acts::Navigator
to disable the boundary check on layer resolval. It also adds an integration test that does check exactely this in a simple cuboid geometry. @pbutti could you check if this solves your problem?cc @paulgessinger @asalzburger
should fix #1596