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

Surf model #300

Merged
merged 2 commits into from
Aug 14, 2024
Merged

Surf model #300

merged 2 commits into from
Aug 14, 2024

Conversation

agheata
Copy link
Contributor

@agheata agheata commented Aug 13, 2024

This re-enables the surface model if VECGEOM_USE_SURF is detected in VecGeom, and ADEPT_USE_SURF is ON (default OFF)

This triggers using AdePTNavigator = SurfNavigator which gets used for the geometry calls.
Behind SurfNavigator we use for now vgbrep::Navigator loop navigator for the Locate and Safety calls, and vgbrep::BVHSurfNavigator for ComputeStep.

Using ADEPT_USE_SURF_SINGLE additionally triggers the surface model with a single precision. When available, this flag will be cross-checked against a corresponding flag in VecGeom.

@phsft-bot
Copy link

Can one of the admins verify this patch?

@agheata agheata marked this pull request as draft August 13, 2024 13:59
@agheata agheata force-pushed the surf_model branch 2 times, most recently from ac0ecd8 to a3d7dab Compare August 13, 2024 14:27
When VECGEOM_USE_SURF is detected and ADEPT_USE_SURF is set, the default navigator becomes SurfNavigator, which dispatches to BVHSurfNavigator.

Additionally, setting ADEPT_USE_SURF_SINGLE uses the surface model in single precision (when this feature will be available)
 * AdePT must be configured in Debug mode and DEBUG_SINGLE_THREAD=ON

 * Added colors to cmake messages.
@agheata agheata marked this pull request as ready for review August 14, 2024 13:08
@agheata
Copy link
Contributor Author

agheata commented Aug 14, 2024

We merge this because it can process all geometries without overlaps, but we know that there are still fixes to be done to be able to validate the model.

@agheata agheata merged commit 65a325d into apt-sim:master Aug 14, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants