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

Detector Parameters Update: Integrated DIRC/MPGD #568

Closed
Chao1009 opened this issue Oct 12, 2023 · 0 comments
Closed

Detector Parameters Update: Integrated DIRC/MPGD #568

Chao1009 opened this issue Oct 12, 2023 · 0 comments
Assignees
Labels
topic: PID Particle identification

Comments

@Chao1009
Copy link
Contributor

A series of smaller issues from #552
det: new value from 2023/09/27 parameter table
sim: current simulation value as of 2023/10/11
template_var: from which we extract the sim value in compact files. Empty means we do not know.
stat: Correct, Missing, or Mismatched

Please implement the new detector parameters, and try to fill in the template_var for the Missing ones if you think it's important for future simulation/design comparison.

Here is the detailed report for this detector subsystem:

CENTRAL DETECTOR, Integrated DIRC/MPGD Detector, : 
                           det    sim            template_var        stat
Length (cm)              488.0    NaN                   Empty     Missing
Inner Radius (cm)         72.5   70.0           {{DIRC_rmin}}  Mismatched
Outer Radius (cm)        108.0   73.0           {{DIRC_rmax}}  Mismatched
Offset from Center (cm) -273.0    NaN                   Empty     Missing
Physical Start (cm)     -303.0 -287.0  -{{DIRCBackward_zmax}}  Mismatched
Physical End (cm)        185.0  185.0    {{DIRCForward_zmax}}     Correct

CENTRAL DETECTOR, Integrated DIRC/MPGD Detector, Dirc Bar/MPGD  Support: 
                           det  sim template_var     stat
Length (cm)              458.0  NaN        Empty  Missing
Inner Radius (cm)         73.3  NaN        Empty  Missing
Outer Radius (cm)         81.0  NaN        Empty  Missing
Offset from Center (cm) -273.0  NaN        Empty  Missing
Physical Start (cm)     -273.0  NaN        Empty  Missing
Physical End (cm)        185.0  NaN        Empty  Missing

CENTRAL DETECTOR, Integrated DIRC/MPGD Detector, DIRC Bar Box: 
                           det      sim        template_var        stat
Length (cm)              458.0  110.145  {{DIRCBar_length}}  Mismatched
Inner Radius (cm)         75.5      NaN               Empty     Missing
Outer Radius (cm)         78.6      NaN               Empty     Missing
Offset from Center (cm) -273.0      NaN               Empty     Missing
Physical Start (cm)     -273.0      NaN               Empty     Missing
Physical End (cm)        185.0      NaN               Empty     Missing

CENTRAL DETECTOR, Integrated DIRC/MPGD Detector, DIRC Readout Support: 
                           det   sim            template_var     stat
Length (cm)               30.0  30.0  {{DIRCReadout_length}}  Correct
Inner Radius (cm)         73.3   NaN                   Empty  Missing
Outer Radius (cm)        108.0   NaN                   Empty  Missing
Offset from Center (cm) -273.0   NaN                   Empty  Missing
Physical Start (cm)     -303.0   NaN                   Empty  Missing
Physical End (cm)       -273.0   NaN                   Empty  Missing

CENTRAL DETECTOR, Integrated DIRC/MPGD Detector, Inner MPGD: 
                           det  sim template_var     stat
Length (cm)              339.0  NaN        Empty  Missing
Inner Radius (cm)         72.5  NaN        Empty  Missing
Outer Radius (cm)         75.0  NaN        Empty  Missing
Offset from Center (cm)    5.0  NaN        Empty  Missing
Physical Start (cm)     -164.5  NaN        Empty  Missing
Physical End (cm)        174.5  NaN        Empty  Missing
@Chao1009 Chao1009 added topic: tracking topic: PID Particle identification and removed topic: PID Particle identification topic: tracking labels Oct 12, 2023
@niwgit niwgit mentioned this issue Oct 24, 2023
7 tasks
github-merge-queue bot pushed a commit that referenced this issue Nov 16, 2023
### Briefly, what does this PR introduce?
Updates the outer MPGD barrel layer (the one integrated with the DIRC)
to match the detector parameter table

### What kind of change does this PR introduce?
- [x ] Bug fix (issue #568 )
- [ ] New feature (issue #__)
- [ ] Documentation update
- [ ] Other: __

### Please check if this PR fulfills the following:
- [ ] Tests for the changes have been added
- [ ] Documentation has been added / updated
- [x ] Changes have been communicated to collaborators

### Does this PR introduce breaking changes? What changes might users
need to make to their code?
This will probably cause an overlap with the DIRC volume, since the
updated DIRC volume is not accounted for in this branch.
### Does this PR change default behavior?
Will likely trigger overlap errors between the DIRC and MPGD layer
github-merge-queue bot pushed a commit that referenced this issue Dec 9, 2023
### Briefly, what does this PR introduce?
This implements following parameters for DIRC.
- Inner radius of 75.5 cm and outer radius of 78.6 cm for the DIRC bar
box. The center of bar is at the average of those two values (r = 77.05
cm).

![dirc_r](https://github.com/eic/epic/assets/23059941/c9c46b40-949b-45f1-b4d8-07194b3ffeac)

- DIRC readout starts at z = -303 cm. 
- Bar box ends at z = 185 cm. DIRC bar ends at z = 183 cm.

![dirc_z](https://github.com/eic/epic/assets/23059941/664a9ca1-c809-4621-a005-0185ad18557b)

### What kind of change does this PR introduce?
- [x] Bug fix (issue #568)
- [ ] New feature (issue #__)
- [ ] Documentation update
- [ ] Other: __

### Please check if this PR fulfills the following:
- [ ] Tests for the changes have been added
- [ ] Documentation has been added / updated
- [ ] Changes have been communicated to collaborators

### Does this PR introduce breaking changes? What changes might users
need to make to their code?

### Does this PR change default behavior?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: PID Particle identification
Projects
Development

No branches or pull requests

3 participants