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

Add detector (mis)alignment info to detector geometry #1363

Open
ShujieL opened this issue Apr 4, 2024 · 1 comment
Open

Add detector (mis)alignment info to detector geometry #1363

ShujieL opened this issue Apr 4, 2024 · 1 comment
Labels
topic: tracking Relates to tracking reconstruction

Comments

@ShujieL
Copy link
Contributor

ShujieL commented Apr 4, 2024

Is your feature request related to a problem? Please describe.
As mentioned by @mposik1983 in the tracking meeting on April 4, 2024, we will need to include detector alignment info for a realistic tracking study.

Describe the solution you'd like
No existing effort within ePIC as I know of. Below are suggestions from @osbornjd based on his sPHENIX experience:

we need to develop our own custom detector element class that interfaces with Acts. This will be called instead of the default ACTS based detector element class and will contain the necessary code to determine what the local to global transform should look like. We then need to also create our own custom class that holds the transformation information. Here's the sPHENIX example detector element class. We then have a class which inherits from the Acts::GeometryContext that holds the relevant transform information here. This probably doesn't need to be an inheritance structure thought, since Acts::GeometryContext is just an alias for std::any

Describe alternatives you've considered
N/A
Additional context
The actual approach needs to be discussed more in the near future.

@ShujieL ShujieL added the topic: tracking Relates to tracking reconstruction label Apr 4, 2024
@veprbl
Copy link
Member

veprbl commented Apr 22, 2024

I suppose, we'd need to learn to simulate the misalignment to quantify do studies on the effect of it. That might have to go into the geant geometry instead. From what I understand, Joe's considerations are for implementing the corrections, which is question looking far off into the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: tracking Relates to tracking reconstruction
Projects
Development

No branches or pull requests

2 participants