Skip to content
Snippets Groups Projects

fix tracker module eta index calculation

Merged Deion Elgin Fellers requested to merge dfellers/calypso:fixTrackerMapping into master

This MR fixes the tracker module mapping such that it matches the desired mapping found on page 4 here. Where this quote from Dave then explains how the eta and phi indices should be calculated:

It is a rule in the offline software that indices increase in the same direction as the global coordinates in a right-handed system with +z downstream and +y (nearly) opposite gravity. So the eta = -1 corresponds to negative X (to the right, when looking downstream), and eta = +1 corresponds to positive X (to the left, when looking downstream). Similarly, the phi module index increases in the upward vertical direction, with phi = 0 closest to the floor.

This results in the correct mapping (module,phi,eta) being: (0,3,1) (1,2,1) (2,1,1) (3,0,1) (4,3,-1) (5,2,-1) (6,1,-1) (7,0,-1)

The old code was calculating the eta index wrong and was thus flipping the tracker modules over X and inverting the X direction of the tracker. It was also first converting the online module number to an offline module number (a relic of how the code worked years ago) before calculating the phi and eta indices. In this MR, i remove the useless and confusing conversion and just calculate the indices directly from the online module number.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
Please register or sign in to reply
Loading