Okay, maybe we can delete this code altogether, because I don't think these values are used at all. I had to bring it up because it was hard to tell how correct your calculation is correct. If you agree, we can remove either in this PR or in a follow up.
Originally posted by @veprbl in #1014 (comment)
This refers to lines
|
// Compute the inner (i.e. thickness until mid-sensitive-volume) and |
|
// outer (from mid-sensitive-volume to top) |
|
// thicknesses that need to be assigned to the tracking surface |
|
// depending on wether the support is above or below the sensor. |
|
double inner_thickness, outer_thickness; |
|
if (sensitiveVolumeSet == 1) { |
|
inner_thickness = thickness_so_far + comp_thickness / 2; |
|
outer_thickness = total_thickness - thickness_so_far - comp_thickness / 2; |
|
} else if (nSensitives == 0) { |
|
inner_thickness = thickness_so_far + comp_thickness / 2; |
|
outer_thickness = comp_thickness / 2; |
|
} else if (nSensitives == 4) { |
|
inner_thickness = comp_thickness / 2; |
|
outer_thickness = total_thickness - thickness_so_far - comp_thickness / 2; |
|
} else { |
|
inner_thickness = outer_thickness = comp_thickness / 2; |
|
} |
|
printout(DEBUG, "BarrelPlanarMPGDTracker_geo", "Sensitive surface @ R = %.4f (%.4f,%.4f) cm", |
|
(thickness_sum + comp_thickness / 2) / cm, inner_thickness / cm, |
|
outer_thickness / cm); |
which we don't believe these values are used, and I can't tell if the values calculated correctly.
Okay, maybe we can delete this code altogether, because I don't think these values are used at all. I had to bring it up because it was hard to tell how correct your calculation is correct. If you agree, we can remove either in this PR or in a follow up.
Originally posted by @veprbl in #1014 (comment)
This refers to lines
epic/src/BarrelPlanarMPGDTracker_geo.cpp
Lines 217 to 236 in 0963404
which we don't believe these values are used, and I can't tell if the values calculated correctly.