Skip to content

Conversation

@pillot
Copy link
Collaborator

@pillot pillot commented Feb 19, 2025

The position of the anode wires in the local coordinate system of the quadrants of station 1 is different w.r.t. the other stations. For station 1, the origin of the system is on top of a wire, while it is shifted by half the pitch for the other stations.

I observed that by looking at the cluster charge asymmetry between the 2 cathodes as a function of the cluster position w.r.t. the closest wire. The pattern was inverted for station 1 w.r.t. other stations and w.r.t. the expected pattern.

I also checked with Christophe on a picture of an open quadrant of station 1. The position of the wires w.r.t. the center of the pads on the non-bending cathode was shifted by half the pitch in MC.

I checked that the effect is negligible on the efficiency and resolution in MC, although it has some effect on the number of pads fired, mostly on the non-bending cathode.

@pillot pillot requested review from a team and shahor02 as code owners February 19, 2025 19:05
@github-actions
Copy link
Contributor

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-PbPb-apass1
async-2024-ppRef-apass1

@pillot pillot merged commit 4ed4dde into AliceO2Group:dev Feb 20, 2025
15 checks passed
@pillot pillot deleted the fix-st1-wire-position branch February 20, 2025 09:56
alcaliva pushed a commit that referenced this pull request Mar 4, 2025
alcaliva pushed a commit that referenced this pull request Mar 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants