Skip to content

fix: some propagation bugs#1079

Merged
niermann999 merged 1 commit intoacts-project:mainfrom
niermann999:feat-overlaps-handling
Oct 27, 2025
Merged

fix: some propagation bugs#1079
niermann999 merged 1 commit intoacts-project:mainfrom
niermann999:feat-overlaps-handling

Conversation

@niermann999
Copy link
Collaborator

@niermann999 niermann999 commented Oct 23, 2025

Fixes several small things:

  • some renaming and reshuffling in the surface and volume (descriptor) classes
  • fixes the momentum value printout in a material interactor log message
  • makes the number of stddev to assume for the mask tolerance estimation configurable in the propagation init
  • has the propagator supervise if the track path length is actually advancing and aborting if it does not
  • remove the propagate_sync method, which is no longer used
  • clean up the propagation debug printing
  • fixes the distance criterion for an overlap in the stepper, aborters and navigation policies
  • fixes an assertion in the builder unittests, which is triggered if a volume is built without containing any surfaces

@niermann999 niermann999 force-pushed the feat-overlaps-handling branch from b8cc5ed to 6b1857b Compare October 24, 2025 12:08
@niermann999 niermann999 force-pushed the feat-overlaps-handling branch from 6b1857b to a4466c4 Compare October 26, 2025 17:33
@sonarqubecloud
Copy link

@niermann999 niermann999 added bug Something isn't working cleanup Benign code cleanup labels Oct 27, 2025
@niermann999 niermann999 merged commit 1b9b579 into acts-project:main Oct 27, 2025
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working cleanup Benign code cleanup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants