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

test: Assert onSurface in Navigator #3533

Merged
merged 10 commits into from
Aug 25, 2024

Conversation

andiwand
Copy link
Contributor

Adds asserts to the Navigator to be sure that we are on the surface we expect to be.

This came up in #3437 and I pulled this out to test it separately.

@andiwand andiwand added this to the next milestone Aug 22, 2024
@github-actions github-actions bot added the Component - Core Affects the Core module label Aug 22, 2024
Copy link

github-actions bot commented Aug 22, 2024

📊: Physics performance monitoring for 842cf3a

Full contents

physmon summary

@andiwand andiwand marked this pull request as ready for review August 22, 2024 11:54
benjaminhuth
benjaminhuth previously approved these changes Aug 22, 2024
Copy link
Member

@benjaminhuth benjaminhuth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Only one small comment, feel free to ignore!
(Funny, I was also about to propose a PR that adds the onSurfaceTolerance to the isOnSurface function).

Core/include/Acts/Surfaces/Surface.hpp Outdated Show resolved Hide resolved
@kodiakhq kodiakhq bot removed the automerge label Aug 23, 2024
Copy link
Contributor

kodiakhq bot commented Aug 23, 2024

This PR currently has a merge conflict. Please resolve this and then re-add the automerge label.

Copy link

sonarcloud bot commented Aug 25, 2024

@kodiakhq kodiakhq bot merged commit 2cfa5f7 into acts-project:main Aug 25, 2024
42 checks passed
@acts-project-service acts-project-service added the Breaks Athena build This PR breaks the Athena build label Aug 25, 2024
@andiwand andiwand deleted the navigator-assert-on-surface branch August 25, 2024 18:01
@paulgessinger paulgessinger removed this from the next milestone Aug 26, 2024
@paulgessinger paulgessinger added this to the v36.2.0 milestone Aug 26, 2024
@acts-project-service acts-project-service added the Fails Athena tests This PR causes a failure in the Athena tests label Aug 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Breaks Athena build This PR breaks the Athena build Component - Core Affects the Core module Fails Athena tests This PR causes a failure in the Athena tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants