-
Notifications
You must be signed in to change notification settings - Fork 45
Demonstrate stuck particle in ATLAS #1914
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
base: develop
Are you sure you want to change the base?
Conversation
Test summary 4 559 files 7 281 suites 13m 18s ⏱️ For more details on these failures, see this check. Results for commit baf2576. ♻️ This comment has been updated with latest results. |
mrguilima
left a comment
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.
Looks good to me.
Thanks!
|
@SethJ, using the master of today I've checked: which traverses the geometry w/o errors: so this probably does not hit the volume you expect? |
|
Thanks for looking @agheata ; the units I posted were cm not mm. Could these be the discrepancy? |
Likely. Running with: I get (printing volume names now): Which starts from a different volume, but there are no errors. |
|
@agheata Does your build have CUDA enabled? Our CI seems to suggest that, somehow, we no longer see the broken behavior on CPU when CUDA is enabled in vecgeom. |
I tried with CUDA disabled, and I am getting the same. Can you try on your side: I could not try with CUDA OFF and SURF OFF because the raytracer does not run in this case due to some missing protection. |
This illustrates the stuck track found by @drbenmorgan in ATLAS documented in #1902 . Because of how VecGeom 1.x determines boundary crossings, it gets stuck when near a plane at a grazing angle. All of ORANGE, Geant4, and VecGeom surface (current master) handle the crossing without error (though vecgeom surface disagrees with Geant4 and VecGeom on some of the safety distances in the outer polycone).
Call sequence to VecGeom:
@agheata @SeverinDiederichs This can probably be reproduced in a VecGeom unit test using the included GDML and raytracing from
{23.51934584635, 17.141066715148, 344.45000092904}along{0.5784236876658104, 0.8157365000698582, -9.290358099212079e-7}.I haven't tried vecgeom 2 with surface disabled.