Skip to content

Commit 31b7c62

Browse files
sawenzelclaude
andcommitted
Place the vacuum mothers of the RB24/3 copper tubes and the RB24 valve flanges
This fixes air instead of vacuum inside the A-side beam pipe. - RB24/3 built the vacuum mother voRB243CuTubeM but placed the bare copper tube, so its 2 x 298 cm bore was cave air. - The valve placed RB24ValveFl instead of its vacuum mother RB24ValveFlI, leaving 2 x 1.5 cm of air. - Both now place the mother. The Run 4 pipe gets the same RB24/3 fix. - The mothers keep the outer shape of the parts they replace, so no new overlaps appear. - The visibility call now hides voRB243CuTubeM instead of voRB24CuTubeM. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
1 parent 7473330 commit 31b7c62

2 files changed

Lines changed: 5 additions & 5 deletions

File tree

‎Detectors/Passive/src/Pipe.cxx‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1277,8 +1277,8 @@ void Pipe::ConstructGeometry()
12771277
// Position all volumes
12781278
Float_t y0;
12791279
TGeoVolumeAssembly* voRB24ValveMo = new TGeoVolumeAssembly("RB24ValveMo");
1280-
voRB24ValveMo->AddNode(voRB24ValveFl, 1, new TGeoTranslation(0., 0., -7.5 / 2. + kRB24ValveFlD / 2.));
1281-
voRB24ValveMo->AddNode(voRB24ValveFl, 2, new TGeoTranslation(0., 0., +7.5 / 2. - kRB24ValveFlD / 2.));
1280+
voRB24ValveMo->AddNode(voRB24ValveFlI, 1, new TGeoTranslation(0., 0., -7.5 / 2. + kRB24ValveFlD / 2.));
1281+
voRB24ValveMo->AddNode(voRB24ValveFlI, 2, new TGeoTranslation(0., 0., +7.5 / 2. - kRB24ValveFlD / 2.));
12821282
y0 = -21.5;
12831283
voRB24ValveMo->AddNode(voRB24ValveBoM, 1, new TGeoTranslation(0., y0 + kRB24ValveBoWy / 2., 0.));
12841284
y0 += kRB24ValveBoWy;
@@ -1691,15 +1691,15 @@ void Pipe::ConstructGeometry()
16911691

16921692
TGeoVolume* voRB243CuTubeM =
16931693
new TGeoVolume("voRB243CuTubeM", new TGeoTube(0., kRB24CuTubeRo, kRB243CuTubeL / 2.), kMedVacNF);
1694-
voRB24CuTubeM->SetVisibility(0);
1694+
voRB243CuTubeM->SetVisibility(0);
16951695
TGeoVolume* voRB243CuTube =
16961696
new TGeoVolume("voRB243CuTube", new TGeoTube(kRB24CuTubeRi, kRB24CuTubeRo, kRB243CuTubeL / 2.), kMedCuNF);
16971697
voRB243CuTubeM->AddNode(voRB243CuTube, 1, gGeoIdentity);
16981698

16991699
TGeoVolumeAssembly* voRB243 = new TGeoVolumeAssembly("RB243");
17001700
TGeoVolumeAssembly* voRB243A = new TGeoVolumeAssembly("RB243A");
17011701

1702-
voRB243A->AddNode(voRB243CuTube, 1, gGeoIdentity);
1702+
voRB243A->AddNode(voRB243CuTubeM, 1, gGeoIdentity);
17031703
z = -kRB243CuTubeL / 2 + kRB24CuTubeFL / 2.;
17041704
voRB243A->AddNode(voRB24CuTubeF, 5, new TGeoTranslation(0., 0., z));
17051705
z = +kRB243CuTubeL / 2 - kRB24CuTubeFL / 2.;

‎Detectors/Passive/src/PipeRun4.cxx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1568,7 +1568,7 @@ void PipeRun4::ConstructGeometry()
15681568
TGeoVolumeAssembly* voRB243 = new TGeoVolumeAssembly("RB243");
15691569
TGeoVolumeAssembly* voRB243A = new TGeoVolumeAssembly("RB243A");
15701570

1571-
voRB243A->AddNode(voRB243CuTube, 1, gGeoIdentity);
1571+
voRB243A->AddNode(voRB243CuTubeM, 1, gGeoIdentity);
15721572
z = -kRB243CuTubeL / 2 + kRB24CuTubeFL / 2.;
15731573
voRB243A->AddNode(voRB24CuTubeF, 5, new TGeoTranslation(0., 0., z));
15741574
z = +kRB243CuTubeL / 2 - kRB24CuTubeFL / 2.;

0 commit comments

Comments
 (0)