Move cables and staves in strip barrel into volumes
Assemblies are groups of volumes without a defined bounding solid. When placing multiple assemblies into a volume, only the totality of all their constituents must not overlap. However, when importing into Geant4, assemblies are removed and the placements are duplicated to allow efficient navigation between non-overlapping siblings.
For our case, this essentially led to a completely flattened tree and resulted in LongStripBarrel1
having a total of 21840 daughter volumes. After the two changes in this merge request, the staves are placed into proper volumes and the barrel volumes have up to 150 daughters at most. The new volume with the most daughters is now LongStripEndcapN
with 7356 daughter volumes, which will be addressed in future changes.
Merge request reports
Activity
Just to give an idea why this is important: The time for
run-ddsim
is reduced from 2m26s to 1m29s and forrun-material-composition
from 24m12s to 14m2s (comparing against the pipeline in !49 (merged)). I'm a bit surprised though that there are measurable differences in some of the material composition plots, the geometry was supposed to stay identical...- Resolved by Andreas Salzburger
@pagessin - to be able to merge this, do we need to update the checks?
added 1 commit
- e34efdc8 - Update reference for material composition check