Skip to content

ALICE3: Improve beampipe geometry and fix position#5548

Merged
sawenzel merged 3 commits into
AliceO2Group:devfrom
mconcas:fix-a3ip-pos
Mar 5, 2021
Merged

ALICE3: Improve beampipe geometry and fix position#5548
sawenzel merged 3 commits into
AliceO2Group:devfrom
mconcas:fix-a3ip-pos

Conversation

@mconcas
Copy link
Copy Markdown
Collaborator

@mconcas mconcas commented Feb 24, 2021

LHC line is displaced wrt LEP magnet of 30 cm. Translating Pipe accordingly.

@mconcas
Copy link
Copy Markdown
Collaborator Author

mconcas commented Feb 24, 2021

image

@mconcas mconcas closed this Feb 25, 2021
@mconcas mconcas reopened this Feb 25, 2021
@mconcas mconcas changed the title Alice3: fix beampipe Y coordinate [WIP] Alice3: fix beampipe Y coordinate Feb 26, 2021
@mconcas
Copy link
Copy Markdown
Collaborator Author

mconcas commented Feb 26, 2021

moved PR to WIP state as it will be added more

@mconcas mconcas requested a review from a team as a code owner February 26, 2021 15:15
@mconcas mconcas changed the title [WIP] Alice3: fix beampipe Y coordinate [WIP] Alice3: Improve beampipe geometry and fix position Feb 26, 2021
@mconcas
Copy link
Copy Markdown
Collaborator Author

mconcas commented Feb 28, 2021 •
edited
Loading

Outer pipe has to be filled with vacuum. There we have also TRK layers, which we don't want to depend on the pipe volume.
Strategy used here is to use a composite shape where shapes of TRK layers are subtracted to the vacuum volume.
For A3PIP-only simulations, we don't want TRK's shade.
Eventually, we will depend on some information passed from the outside.

With TRK:
image
Without TRK:
image

ligh-green is the vacuum.

Thanks to @mario6829 for guiding me in the troubleshooting.

@mconcas mconcas changed the title [WIP] Alice3: Improve beampipe geometry and fix position ALICE3: Improve beampipe geometry and fix position Feb 28, 2021
qgp
qgp reviewed Mar 1, 2021
View reviewed changes
Comment thread Detectors/Upgrades/ALICE3/Passive/include/Alice3DetectorsPassive/Pipe.h Outdated
Comment thread Detectors/Upgrades/ALICE3/Passive/src/Pipe.cxx Outdated
Comment thread Detectors/Upgrades/ALICE3/Passive/include/Alice3DetectorsPassive/Pipe.h Outdated
qgp
qgp reviewed Mar 3, 2021
View reviewed changes
Comment thread Detectors/Upgrades/ALICE3/Passive/include/Alice3DetectorsPassive/Pipe.h Outdated
qgp
qgp previously approved these changes Mar 3, 2021
View reviewed changes
@sawenzel sawenzel merged commit 610b926 into AliceO2Group:dev Mar 5, 2021
@mconcas mconcas deleted the fix-a3ip-pos branch April 24, 2021 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@qgp qgp qgp approved these changes

@marcovanleeuwen marcovanleeuwen Awaiting requested review from marcovanleeuwen

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mconcas @qgp @sawenzel