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

Improve geometry testing #1652

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

sethrj
Copy link
Member

@sethrj sethrj commented Mar 4, 2025

This pulls another couple of small changes from #1422 . The geometry heuristic test now guarantees reproducibility between CPU and GPU. The VecGeom test is now somewhat simplified. Finally the ScopedLimitSaver used by VecgeomParams is now cleaned up a bit.

@sethrj sethrj added documentation Documentation, examples, tests, and CI minor Minor internal changes or fixes geometry Geometry-related features labels Mar 4, 2025
@sethrj sethrj requested a review from mrguilima March 4, 2025 22:31
Copy link

github-actions bot commented Mar 4, 2025

Test summary

 4 180 files   6 435 suites   13m 38s ⏱️
 1 741 tests  1 730 ✅ 11 💤 0 ❌
22 407 runs  22 324 ✅ 83 💤 0 ❌

Results for commit 054e8fe.

♻️ This comment has been updated with latest results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Documentation, examples, tests, and CI geometry Geometry-related features minor Minor internal changes or fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant