Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Overlaps of cavern with muon shield and tracking stations, gaps in cavern #599

Closed
olantwin opened this issue Feb 10, 2025 · 4 comments · Fixed by #630
Closed

Overlaps of cavern with muon shield and tracking stations, gaps in cavern #599

olantwin opened this issue Feb 10, 2025 · 4 comments · Fixed by #630

Comments

@olantwin
Copy link
Contributor

olantwin commented Feb 10, 2025

Introduced by #592.

=== Overlaps for FAIRGeom ===
 = Overlap ov00000: cave/Cavern_1 overlapping cave/MuonShieldArea_1/Magn4_MiddleMagL_1 ovlp=20.4398
 = Overlap ov00001: cave/Cavern_1 overlapping cave/MuonShieldArea_1/Magn4_MiddleMagR_1 ovlp=20.4398
 = Overlap ov00002: cave/Cavern_1 overlapping cave/MuonShieldArea_1/Magn4_MagRetL_1 ovlp=20.4398
 = Overlap ov00003: cave/Cavern_1 overlapping cave/MuonShieldArea_1/Magn4_MagRetR_1 ovlp=20.4398
 = Overlap ov00004: cave/Cavern_1 overlapping cave/MuonShieldArea_1/Magn4_MagTopLeft_1 ovlp=20.4398
 = Overlap ov00005: cave/Cavern_1 overlapping cave/MuonShieldArea_1/Magn4_MagTopRight_1 ovlp=20.4398
 = Overlap ov00006: cave/Cavern_1 overlapping cave/MuonShieldArea_1/Magn4_MagBotLeft_1 ovlp=20.4398
 = Overlap ov00007: cave/Cavern_1 overlapping cave/Tr4_4 ovlp=8.28469
 = Overlap ov00008: cave/Cavern_1 overlapping cave/Tr1_1 ovlp=5.89438
 = Overlap ov00009: cave/Cavern_1 overlapping cave/Tr2_2 ovlp=5.89438

This is also quite obvious visually:

Image

Image

@olantwin olantwin marked this as a duplicate of #600 Feb 10, 2025
@olantwin olantwin changed the title Overlaps of cavern with muon shield and tracking stations Overlaps of cavern with muon shield and tracking stations, gaps in cavern Feb 10, 2025
@olantwin olantwin linked a pull request Feb 11, 2025 that will close this issue
@olantwin
Copy link
Contributor Author

@wcmartylee Please also check with the SST group, what the plan is for the straw trackers. Not sure whether the trench needs to move or changes to the tracker are needed.

@olantwin
Copy link
Contributor Author

Overlap with muon shield will be fixed by #609

@olantwin
Copy link
Contributor Author

The change to the cavern was premature and will be reverted, so this issue no longer applies.

@olantwin olantwin closed this as not planned Won't fix, can't repro, duplicate, stale Feb 12, 2025
@olantwin
Copy link
Contributor Author

Reopening as the decision on the revert will be taken Monday.

@olantwin olantwin reopened this Feb 13, 2025
@wcmartylee wcmartylee linked a pull request Feb 20, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants