Skip to content

Do not keep physical nodes when aligning the o2-sim geometry - #16119

Open
sawenzel wants to merge 1 commit into
AliceO2Group:devfrom
sawenzel:sim-align-drop-physical-nodes
Open

sawenzel wants to merge 1 commit into
AliceO2Group:devfrom
sawenzel:sim-align-drop-physical-nodes

Conversation

@sawenzel

Copy link
Copy Markdown
Collaborator

This lowers the memory of o2-sim by deleting each TGeo physical node right after its alignment. They are
not needed during simulation.

  • Applying the alignment created one TGeoPhysicalNode per alignable volume (29582), kept for the whole job.
  • GeometryManager::applyAlignment and Aligner::applyAlignment get an option keepPhysicalNodes, true by default.
  • o2-sim sets it to false, since it needs no original matrices; reconstruction and alignment are unchanged.

The effect is roughly ~80MB.

This lowers the memory of o2-sim by deleting each TGeo physical node right after its alignment.

- Applying the alignment created one TGeoPhysicalNode per alignable volume (29582), kept for the whole job.
- GeometryManager::applyAlignment and Aligner::applyAlignment get an option keepPhysicalNodes, true by default.
- o2-sim sets it to false, since it needs no original matrices; reconstruction and alignment are unchanged.
@sawenzel
sawenzel requested a review from shahor02 as a code owner November 13, 2026 07:57
@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2/fullCI_slc9 for 47f932a at 2026-11-13 09:00:

No log files found

Full log here.

This branch has not been deployed

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants