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.

Sign up for GitHub

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

Jump to bottom

dRICH: implement dual mirrors #19

Open
c-dilks opened this issue Jul 27, 2022 · 1 comment · May be fixed by #380
Open

dRICH: implement dual mirrors #19

c-dilks opened this issue Jul 27, 2022 · 1 comment · May be fixed by #380
Assignees
@chchatte92
Labels
enhancement New feature or request

Comments

@c-dilks
Copy link
Member

c-dilks commented Jul 27, 2022

Describe the solution you'd like
Implement dual or multi mirrors (per sector) in the dRICH. This will allow us to place sensors in a more confined area.

For a first attempt at this in DD4hep (athena), see https://eicweb.phy.anl.gov/EIC/detectors/athena/-/merge_requests/260
@c-dilks c-dilks self-assigned this Jul 27, 2022
@c-dilks c-dilks moved this to Todo in dRICH Simulations Aug 4, 2022
@veprbl veprbl added the enhancement New feature or request label Aug 6, 2022
@c-dilks c-dilks assigned chchatte92 and unassigned c-dilks Feb 13, 2023
@chchatte92
Copy link
Member

Hi Chris, I have created the branch and will implement the changes there and then after testing them we will decide.
Here is the branch:
https://github.com/eic/epic/tree/dual_mirror_implementation

@c-dilks c-dilks linked a pull request Feb 28, 2023 that will close this issue
Dual mirror implementation #380
Draft
@c-dilks c-dilks moved this from Todo to In Progress in dRICH Simulations Mar 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chchatte92 chchatte92

Labels
enhancement New feature or request
Projects
dRICH Simulations
Status: In Progress
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Dual mirror implementation eic/epic
3 participants
@veprbl @c-dilks @chchatte92