-
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: Fully configurable layer envelopes #1751
feat: Fully configurable layer envelopes #1751
Conversation
📊 Physics performance monitoring for 8e0213dFull report VertexingSeedingCKFAmbiguity resolutionTruth tracking (Kalman Filter)Truth tracking (GSF) |
Codecov Report
@@ Coverage Diff @@
## main #1751 +/- ##
=======================================
Coverage 49.14% 49.14%
=======================================
Files 395 395
Lines 21800 21802 +2
Branches 9953 9953
=======================================
+ Hits 10713 10715 +2
Misses 4187 4187
Partials 6900 6900
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
This PR is to allow for fully configure the CuboidVolumeBuilder to define the envelope of the various layers. Allowing Navigation layers to be wider than just the size of the sensitive surfaces can help with problems seen in issue acts-project#1596 to account for the fact that the Navigator will extrapolate linearly from the last state, not taking into account that the track will bend. This is only relevant in telescope geometries.
This PR is to allow for fully configure the CuboidVolumeBuilder to define the envelope of the various layers.
Allowing Navigation layers to be wider than just the size of the sensitive surfaces can help with problems seen in issue
#1596 to account for the fact that the Navigator will extrapolate linearly from the last state, not taking into account that the track will bend. This is only relevant in telescope geometries.