Closed EinarElen closed 1 year ago
Ok, I've worked out all the issues now and tested them thoroughly. What really threw me off is that the entire Hcal geometry is offset in Y by 19.05 mm. I'm not sure why this is, but it can be corrected for relatively straight-forwardly. I'll make a PR with the patches for v14.
Describe the bug As Bertrand reported from his work on the v14 PN samples, the side Hcal position reconstruction isn't working right. The side hcal position reconstruction is taken directly from the Hcal geometry so the correct location to look for issues will be in building the side hcal center position map.
I've walked through all of the code leading up to this part, including the GDML and python configuration and I believe those are correct. I have also checked that the conversion from copy number to HcalID is correct on the simulation level.
Here are two sketches I made that I think is helpful to have as reference. Obviously, nothing here is to scale. A layer here corresponds to one absorber plate, two airgaps and one scintillator layer. Overview of a single section
Overview of the v14 side hcal from the front. Note the orientation of the X-axis.
Based on this sketch, I've outlined a small set of sanity checks that I'm going to check. Checked are tested and pass, checked and strikethrough are tested and fail at least once.
Top/Bottom
General features
Features for odd bars (along Z)
Bars oriented along the Z axis all have the same Z centerBars within different modules have different Z center, e.g.Bars oriented along the Z axis all have different X center within a given layer (offset by the width of a scintillator bar), but same as bars with the same strip numberAll bars within a given module currently have the same X centerFeatures for even bars (along X)
Bars oriented along the X axis all have the same X center within a given module, but different between modules.Example bars in the same module with different XExample bars in different modules with the same X
Hcal(2,8,10) -> (-765, -642, 765)
Hcal(1,8,8) -> (665, 642, 665)