From 69c16c17ea61bcf09c3f370ae9f9b1b3bcb7a938 Mon Sep 17 00:00:00 2001
From: Nathan Baltzell <baltzell@jlab.org>
Date: Mon, 4 May 2026 16:26:22 -0400
Subject: [PATCH] reduce atan calculations

---
 .../java/org/jlab/rec/cvt/cross/Cross.java    | 36 ++++++++++---------
 1 file changed, 20 insertions(+), 16 deletions(-)

diff --git a/reconstruction/cvt/src/main/java/org/jlab/rec/cvt/cross/Cross.java b/reconstruction/cvt/src/main/java/org/jlab/rec/cvt/cross/Cross.java
index fcc5cc01e9..5725ea6f3b 100644
--- a/reconstruction/cvt/src/main/java/org/jlab/rec/cvt/cross/Cross.java
+++ b/reconstruction/cvt/src/main/java/org/jlab/rec/cvt/cross/Cross.java
@@ -616,33 +616,37 @@ public int getSVTCosmicsRegion() {
     @Override
     public int compareTo(Cross arg) {
 
-        int return_val = 0;
-        if(Constants.getInstance().isCosmics) {
+        int return_val;
+        if (Constants.getInstance().isCosmics) {
             int RegComp = this.getSVTCosmicsRegion() < arg.getSVTCosmicsRegion() ? -1 : this.getSVTCosmicsRegion() == arg.getSVTCosmicsRegion() ? 0 : 1;
             int IDComp = this.getId() < arg.getId() ? -1 : this.getId() == arg.getId() ? 0 : 1;
-
             return_val = ((RegComp == 0) ? IDComp : RegComp);
-        } else {
+        }
+        else {
             
-            //int thisreg = (this.getDetector().equalsIgnoreCase("BMT")) ? 3 + bgeom.getLayer( this.getRegion(), this.getDetectorType()) : this.getRegion();
-            //int argreg  = (arg.getDetector().equalsIgnoreCase("BMT"))  ? 3 + bgeom.getLayer( arg.getRegion(), arg.getDetectorType()) : arg.getRegion();
-            int thisreg = this.getOrderedRegion();
-            int argreg  = arg.getOrderedRegion();
-            int RegComp = thisreg < argreg ? -1 : thisreg == argreg ? 0 : 1;
-//            int RegComp = this.getRegion() < arg.getRegion() ? -1 : this.getRegion() == arg.getRegion() ? 0 : 1;
+            int RegComp = this.getOrderedRegion() < arg.getOrderedRegion() ? -1 : 
+                          this.getOrderedRegion() == arg.getOrderedRegion() ? 0 : 1;
             
             // check that is not BMTC for phi comparison
             if( Double.isNaN(arg.getPoint().x())==false &&  Double.isNaN(this.getPoint().x())==false ) {
-            	int PhiComp = this.getPoint0().toVector3D().phi() < arg.getPoint0().toVector3D().phi() ? -1 : this.getPoint0().toVector3D().phi() == arg.getPoint0().toVector3D().phi() ? 0 : 1;
-            
-            	return_val = ((RegComp == 0) ? PhiComp : RegComp);
+                if (RegComp == 0) {
+                    double phi0 = this.getPoint0().toVector3D().phi();
+                    double phi1 = arg.getPoint0().toVector3D().phi();
+                    return_val = phi0 < phi1 ? -1 : phi0 == phi1 ? 0 : 1;
+                }
+                else {
+                    return_val = RegComp;
+                }
             }
             else {
-            	int ZComp = this.getPoint0().z() < arg.getPoint0().z() ? -1 : this.getPoint0().z() == arg.getPoint0().z() ? 0 : 1;
-            	return_val = ((RegComp == 0) ? ZComp : RegComp);
+                if (RegComp == 0) {
+                    return_val = this.getPoint0().z() < arg.getPoint0().z() ? -1 : this.getPoint0().z() == arg.getPoint0().z() ? 0 : 1;
+                }
+                else {
+                    return_val = RegComp;
+                }
             }
         }
-
         return return_val;
     }