Skip to content
This repository has been archived by the owner on Sep 4, 2024. It is now read-only.

Second flange presence in beampipe in forward direction #15

Closed
rymilton opened this issue Aug 23, 2022 · 3 comments · Fixed by #16
Closed

Second flange presence in beampipe in forward direction #15

rymilton opened this issue Aug 23, 2022 · 3 comments · Fixed by #16
Labels
question Further information is requested

Comments

@rymilton
Copy link
Contributor

Is your feature request related to a problem? Please describe.
Should the second flange in the beampipe in the forward direction (z = ~445 cm) be present? I've been told by multiple people in the calorimetry group that this flange is from an older beampipe design. The flange overlaps with the proposed design of the forward HCal insert.

Describe the solution you'd like
Remove the secondary flange.

Describe alternatives you've considered
Maintain the flange and adjust the design of the insert.

Additional context
Image of the beampipe:
image

@rymilton rymilton added the question Further information is requested label Aug 23, 2022
@wdconinc
Copy link
Contributor

The flange can be removed indeed. When it became clear that the flange placement was notional, we didn't have as much urgency to fix this. We just moved it 5 cm to avoid overlap with the drich. The lines 62 through 67 in https://github.com/eic/ip6/blob/master/ip6/central_beampipe.xml#L62 describe that flange.

@wdconinc
Copy link
Contributor

Ah, that's the first flange and you're talking about the second flange, but the same is true there. The other aspect there is that we should also adjust the pipe itself to mate with the B0 magnet, with or without flange.

@rymilton
Copy link
Contributor Author

For the time being, I can submit a pull request that removes both of these flanges by removing lines 63-66 & 69-72. We can then adjust the pipe further for the B0 magnet in another pull request.

@wdconinc wdconinc linked a pull request Aug 24, 2022 that will close this issue
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants