Skip to content

Conversation

@mathieuouillon
Copy link
Collaborator

No description provided.

@mathieuouillon mathieuouillon marked this pull request as ready for review April 8, 2025 22:05
@baltzell
Copy link
Collaborator

baltzell commented Apr 8, 2025

If I understand correctly, the proposal is to make the geometry service s/l/c/o convention match the tracking code rather than the expected/ideal geometry?

@raffaelladevita
Copy link
Collaborator

Mathieu can confirm but I think this PR is updating the AHDC geometry to account for a change in the detector wire numbering specifications that happened some time ago and was not propagated to the software:

  • the current geometry assumes the first wire of each layer is at phi=-90 deg and the wire number decreases for increasing phi;
  • with this PR, wire 1 is at phi=0, and the wire number increases with phi.

@baltzell baltzell merged commit 2f0322b into JeffersonLab:development Apr 9, 2025
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants