From 92d9cbd66ec75b26ebcc4db4df97a8f49f4d41a3 Mon Sep 17 00:00:00 2001
From: Wouter Deconinck <wouter.deconinck@umanitoba.ca>
Date: Wed, 9 Jun 2021 15:39:52 +0000
Subject: [PATCH] Solenoid offset 10 cm in hadron direction

---
 compact/definitions.xml | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/compact/definitions.xml b/compact/definitions.xml
index e24d7e2f..aaf4c6ea 100644
--- a/compact/definitions.xml
+++ b/compact/definitions.xml
@@ -294,9 +294,10 @@
        - "rmin", "rmax", "thickness" all refer to the radial direction.
        - "zmin", "zmax", "length", "offset" all refer to the z direction.
        - Positive and Negative are  "EndcapP" and "EndcapN", respectively.
-       - All values are positive. When refering to a negative endcap z position, the values are
+       - All values are positive (except offsets, which are positive in the hadron direction, i.e. towards EndCapP).
+           When refering to a negative endcap z position, the values are
            also positive then negated when required. (Many endcap constructions are mirrored, 
-           hence a postive value.
+           hence a postive value.)
        - We will try to keep the subsystem component name first and the parameter after an "_" (eg, EcalBarrel_length").
            TODO: fix parameter naming to be uniform in the above convention.
 
@@ -311,7 +312,7 @@
     <constant name="Solenoid_length"           value="3840.0*mm"/>
     <constant name="Solenoid_rmin"             value="1600.0*mm"/>
     <constant name="Solenoid_thickness"        value="640*mm"/>
-    <constant name="Solenoid_offset"           value="0.0*mm"/>
+    <constant name="Solenoid_offset"           value="10*mm"/>
 
     <constant name="Solenoid_rmax"             value="Solenoid_rmin + Solenoid_thickness"/>
     <constant name="SolenoidBarrel_rmin"       value="Solenoid_rmin"/>
-- 
GitLab