Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Read new /geometry/shifts/solenoid and /geometry/shifts/target CCDB tables to set target and solenoid positions independently #213

Merged
merged 2 commits into from
Mar 8, 2024

Conversation

raffaelladevita
Copy link
Collaborator

To support runs in which the target is not installed at the center of the solenoid (RG-C, RG-D), new CCDB tables were created under /geometry/shifts to store the x,y,z coordinates of the various volumes.
See https://clasweb.jlab.org/cgi-bin/ccdb/versions?table=/geometry/shifts/solenoid and https://clasweb.jlab.org/cgi-bin/ccdb/versions?table=/geometry/shifts/target

Previously, /geometry/target was used to set the positions of target, solenoid, CTOF and CND and was typically set to -3 cm. Apart from being inaccurate, this was not a problem so far because the actual target position was always equal to the solenoid center. This is not the case for RG-C and RG-D and, since the target position is used by CVT for eloss and selection of track candidates that by default is set to |z_seed-ztarget|<targetlength/2+1cm, using a wrong position leads to inaccurate eloss corrections and, more importantly, loss of good tracks.

In the new implementation:

  • /geometry/shits/target is used by:
    • CVT for eloss and candidates selection,
    • FD tracking to define trajectory planes;
  • /geometry/shifts/solenoid is used by:
    • Magfield to set the shift of the solenoid map,
    • CTOF and CND to set the detector position,
    • EB as a reference point for the RF correction,
    • FT to set the particle vertices.

The last 2, RF and FT, are temporary solutions to be consistent with the RG-C Summer 22 calibrations. These will be changes as follows in the future:

  • EB will not use a reference vz anymore since that can be simply reabsorbed in the RF calibrations;
  • FT will use the target center.

Closes #211

@raffaelladevita raffaelladevita merged commit a387518 into development Mar 8, 2024
67 checks passed
@raffaelladevita raffaelladevita deleted the iss211-positions branch March 8, 2024 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

decouple target and solenoid positions
2 participants