Skip to content

chore: SurfaceMaterial without BinUtility #5304

Draft
asalzburger wants to merge 7 commits intoacts-project:mainfrom
asalzburger:feat-material-wo-binuntility
Draft

chore: SurfaceMaterial without BinUtility #5304
asalzburger wants to merge 7 commits intoacts-project:mainfrom
asalzburger:feat-material-wo-binuntility

Conversation

@asalzburger
Copy link
Copy Markdown
Contributor

@asalzburger asalzburger commented Mar 31, 2026

Quick snapshot based on this branch vs origin/main merge-base, restricted to files touched in this refactor:

  • Files containing BinUtility: 22 -> 11 (-50%)
  • Total BinUtility token occurrences: 118 -> 100 (-18, ~-15%)
  • #include "Acts/Utilities/BinUtility.hpp" files: 15 -> 9 (-40%)
  • Total BinUtility.hpp include lines: 15 -> 9 (-40%)

For the core Material implementation area specifically (Core/include/Acts/Material + Core/src/Material):

  • Files containing BinUtility: 14 -> 11 (-21%)
  • Include lines: 9 -> 7 (-22%)

--- END COMMIT MESSAGE ---

Any further description goes here, @-mentions are ok here!

  • Use a conventional commits prefix: quick summary
    • We mostly use feat, fix, refactor, docs, chore and build types.
  • A milestone will be assigned by one of the maintainers

@asalzburger asalzburger marked this pull request as draft March 31, 2026 13:08
@github-actions github-actions bot added this to the next milestone Mar 31, 2026
@github-actions github-actions bot added Component - Core Affects the Core module Component - Examples Affects the Examples module Component - Plugins Affects one or more Plugins labels Mar 31, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 31, 2026

📊: Physics performance monitoring for ad155e0

Full contents

physmon summary

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component - Core Affects the Core module Component - Examples Affects the Examples module Component - Plugins Affects one or more Plugins

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant