Skip to content
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: Reintroduce branch stopping on measurement in Core CKF #3172

Merged
merged 3 commits into from
May 7, 2024

Conversation

andiwand
Copy link
Contributor

@andiwand andiwand commented May 6, 2024

After removing this in #3102 I reintroduce this functionality here to reduce the changes to a minimum. After #3102 we see some bigger tack finding changes than expected which I suspect are coming from the fact that we effectively stopped most of the branches when reaching a measurement surface.

Having this callback will also be necessary if we want to allow the user to count holes/outliers/measurements in specific geometry regions by themselves.

@andiwand andiwand added this to the next milestone May 6, 2024
@github-actions github-actions bot added Component - Core Affects the Core module Track Finding labels May 6, 2024
@kodiakhq kodiakhq bot merged commit 320253c into acts-project:main May 7, 2024
51 checks passed
@github-actions github-actions bot removed the automerge label May 7, 2024
@andiwand andiwand deleted the feat-ckf-reint-stop-on-meas branch May 7, 2024 09:21
@andiwand andiwand modified the milestones: next, v35.0.0 May 17, 2024
asalzburger pushed a commit to asalzburger/acts that referenced this pull request May 21, 2024
…oject#3172)

After removing this in acts-project#3102 I reintroduce this functionality here to reduce the changes to a minimum. After acts-project#3102 we see some bigger tack finding changes than expected which I suspect are coming from the fact that we effectively stopped most of the branches when reaching a measurement surface.

Having this callback will also be necessary if we want to allow the user to count holes/outliers/measurements in specific geometry regions by themselves.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component - Core Affects the Core module Track Finding
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants