Skip to content

Commit 4746c27

Browse files
authored
Clean up comments on material registration process
Removed commented-out code regarding material registration and its implications.
1 parent 53690be commit 4746c27

1 file changed

Lines changed: 1 addition & 14 deletions

File tree

‎Detectors/Upgrades/ALICE3/TRKFT3/FT3/simulation/src/FT3Module.cxx‎

Lines changed: 1 addition & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -55,20 +55,7 @@ void FT3Module::initialize_materials()
5555
if (siliconMat) {
5656
return;
5757
}
58-
59-
// NOTE: these materials/media used to be registered directly via
60-
// `new TGeoMaterial(...)` / `new TGeoMedium(name, rawId, ...)` with small
61-
// hand-picked ids (1-6). That bypasses o2::base::MaterialManager, whose job
62-
// is to hand out globally-unique medium ids across all detectors. Since the
63-
// 3-argument TGeoMedium constructor also leaves all tracking parameters
64-
// (including ifield) at 0, and Geant4VMC's TG4GeometryManager::
65-
// FillMediumMapFromRoot() maps ALL TGeoMedium objects into one global table
66-
// keyed purely by that raw numeric id (last one processed wins), these raw
67-
// ids collided with other detectors' properly-assigned medium ids (e.g. the
68-
// CAVE air medium that fills the gaps between TRK layers) and silently
69-
// switched the magnetic field off for them. Route through MaterialManager
70-
// instead, like every other detector (incl. FT3's own Detector.cxx) does,
71-
// so ids are safely auto-assigned and never collide.
58+
7259
auto& matmgr = o2::base::MaterialManager::Instance();
7360

7461
int ifield = 2;

0 commit comments

Comments
 (0)