Skip to content

Conversation

@raffaelladevita
Copy link
Collaborator

@raffaelladevita
Copy link
Collaborator Author

Checked once again with data
Plot_04-15-2025_06 16 24_PM

Requires updating current TT to
ahdc.txt
where the last two columns are the x and y of the wires at the upstream end

@RDupre
Copy link
Collaborator

RDupre commented Apr 16, 2025

I am fine with the new numbering, wires of each layers are properly aligned.

I am not sure about the mapping file associated with it. It seems to me it was generated without the cable swaps included (I assume this swap was not an artifact linked with the geometry service and that it should still be included). Was it included in a different way to produce the results presented here?

I also think there is a slight error with the numbering due to the fact that the new definition does not give consistent shift in a given super-layer (by my calculation in SL1, L0 is moved by 28 wires (29 becomes 1), but L1 is only moved by 28). This is due to the change of direction in phi in the count method, the wires not aligned at 0 are not shifted by 180 degrees but less.

I attach what I get from my calculation: AHDCmap_vn.txt

@baltzell baltzell closed this Apr 16, 2025
@baltzell baltzell mentioned this pull request Apr 16, 2025
@baltzell
Copy link
Collaborator

replaced by #571

@baltzell baltzell deleted the ahdc-geo-revert-retry branch May 21, 2025 21:33
@baltzell baltzell added the rg-l label Dec 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants