Moving muonSystemBounds to SequentialVertexFitter

Author: Lovisa

RecoVertex/KalmanVertexFit/interface/KalmanVertexFitter.h

@@ -109,9 +109,6 @@ class KalmanVertexFitter : public VertexFitter<5> {
   edm::ParameterSet defaultParameters() const;
 
   SequentialVertexFitter<5>* theSequentialFitter;
-
-  float muonSystemBoundsRadius{740.};
-  float muonSystemBoundsHalfLength{960.};
 };
 
 #endif

RecoVertex/KalmanVertexFit/src/KalmanVertexFitter.cc

@@ -35,9 +35,8 @@ void KalmanVertexFitter::setup(const edm::ParameterSet& pSet, bool useSmoothing,
                                                         smoother,
                                                         LinearizedTrackStateFactory());
   }
-  if (useMuonSystemBounds) {
-    theSequentialFitter->setTrackerBounds(muonSystemBoundsRadius, muonSystemBoundsHalfLength);
-  }
+  if (useMuonSystemBounds)
+    theSequentialFitter->setMuonSystemBounds();
 }
 
 edm::ParameterSet KalmanVertexFitter::defaultParameters() const {

RecoVertex/VertexTools/interface/SequentialVertexFitter.h

@@ -180,6 +180,11 @@ class SequentialVertexFitter : public VertexFitter<N> {
     trackerBoundsHalfLength = halfLength;
   }
 
+  void setMuonSystemBounds() {
+    trackerBoundsRadius = kMuonSystemBoundsRadius;
+    trackerBoundsHalfLength = kMuonSystemBoundsHalfLength;
+  }
+
 protected:
   /**
    *   Default constructor. Is here, as we do not want anybody to use it.
@@ -245,6 +250,8 @@ class SequentialVertexFitter : public VertexFitter<N> {
   // FIXME using hard-coded tracker bounds as default instead of taking them from geometry service
   float trackerBoundsRadius{112.};
   float trackerBoundsHalfLength{273.5};
+  static constexpr float kMuonSystemBoundsRadius = 740.;
+  static constexpr float kMuonSystemBoundsHalfLength = 960.;
 
   edm::ParameterSet thePSet;
   LinearizationPointFinder* theLinP;