-
Notifications
You must be signed in to change notification settings - Fork 176
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
chore: Remove volumeSignature
from geometry building
#2759
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2759 +/- ##
=======================================
Coverage 49.50% 49.50%
=======================================
Files 474 474
Lines 27068 27068
Branches 12511 12511
=======================================
Hits 13401 13401
Misses 4761 4761
Partials 8906 8906 ☔ View full report in Codecov by Sentry. |
📊: Physics performance monitoring for 454e1d8physmon summary
|
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.
Good job.
Turns out this breaks Athena. Do we revert until next week? |
…project#2759)" This reverts commit 34bd984.
Let's revert until next week, although I really like the cleanup - so let's do it regardless later. |
…2759) Seems to have no purpose anymore in ACTS. I was introduced in the pre-PR time: acts-project@cd4c063 Closes acts-project#1731
…#2764) This reverts commit 34bd984 from acts-project#2759 as it breaks Athena compilation.
Seems to have no purpose anymore in ACTS.
I was introduced in the pre-PR time: cd4c063
Closes #1731