add comments

dudarboh · Oct 1, 2024 · e7d4a2e · e7d4a2e
1 parent da013d4
commit e7d4a2e
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/Tracking/TrackLength/src/TrackLengthUtils.cc b/Tracking/TrackLength/src/TrackLengthUtils.cc
@@ -46,6 +46,8 @@ IMPL::TrackStateImpl TrackLengthUtils::getTrackStateAtHit(MarlinTrk::IMarlinTrac
 
 
 double TrackLengthUtils::getHelixLength(const EVENT::TrackState& ts1, const EVENT::TrackState& ts2){
+    // OPTIMIZE: Are results better with tanL = ts2.getTanLambda(); ?
+    // Or tanL = (ts1+ts2)/2 ? Or some other weighted combination? ...
     double tanL = ts1.getTanLambda();
     double z1 = ts1.getReferencePoint()[2] + ts1.getZ0();
     double z2 = ts2.getReferencePoint()[2] + ts2.getZ0();
@@ -54,7 +56,7 @@ double TrackLengthUtils::getHelixLength(const EVENT::TrackState& ts1, const EVEN
 
 
 double TrackLengthUtils::getHelixLengthOption1(const EVENT::TrackState& ts1, const EVENT::TrackState& ts2){
-    // Suboptimal and not used. Use getHelixLength()
+    // Prooven suboptimal and kept only for reference. Use getHelixLength()
     double omega = std::abs( ts1.getOmega() );
     double z1 = ts1.getReferencePoint()[2] + ts1.getZ0();
     double z2 = ts2.getReferencePoint()[2] + ts2.getZ0();
@@ -70,7 +72,7 @@ double TrackLengthUtils::getHelixLengthOption1(const EVENT::TrackState& ts1, con
 
 
 double TrackLengthUtils::getHelixLengthOption2(const EVENT::TrackState& ts1, const EVENT::TrackState& ts2){
-    // Suboptimal and not used. Use getHelixLength()
+    // Prooven suboptimal and kept only for reference. Use getHelixLength()
     double omega = std::abs( ts1.getOmega() );
     double tanL = std::abs( ts1.getTanLambda() );
     double dPhi = std::abs( ts2.getPhi() - ts1.getPhi() );