diff --git a/Core/include/Acts/TrackFinding/MeasurementSelector.hpp b/Core/include/Acts/TrackFinding/MeasurementSelector.hpp index 74ac4f0034f..08d5e41a452 100644 --- a/Core/include/Acts/TrackFinding/MeasurementSelector.hpp +++ b/Core/include/Acts/TrackFinding/MeasurementSelector.hpp @@ -14,6 +14,7 @@ #include "Acts/EventData/MultiTrajectory.hpp" #include "Acts/EventData/TrackParameters.hpp" #include "Acts/Geometry/GeometryHierarchyMap.hpp" +#include "Acts/Geometry/GeometryIdentifier.hpp" #include "Acts/TrackFinding/CombinatorialKalmanFilterError.hpp" #include "Acts/Utilities/Logger.hpp" #include "Acts/Utilities/Result.hpp" @@ -37,7 +38,7 @@ struct MeasurementSelectorCuts { /// bins in |eta| to specify variable selections std::vector etaBins{}; /// Maximum local chi2 contribution. - std::vector chi2CutOff{std::numeric_limits::max()}; + std::vector chi2CutOff{15}; /// Maximum number of associated measurements on a single surface. std::vector numMeasurementsCutOff{1}; }; @@ -61,11 +62,19 @@ class MeasurementSelector { using Config = Acts::GeometryHierarchyMap; /// @brief Default constructor - MeasurementSelector() = default; - /// @brief Constructor with config and (non-owning) logger + /// + /// This will use the default configuration for the cuts. + MeasurementSelector(); + + /// @brief Constructor with cuts + /// + /// @param cuts The cuts to use + explicit MeasurementSelector(const MeasurementSelectorCuts& cuts); + + /// @brief Constructor with config /// /// @param config a config instance - MeasurementSelector(Config config); + explicit MeasurementSelector(Config config); /// @brief Function that select the measurements compatible with /// the given track parameter on a surface diff --git a/Core/src/TrackFinding/MeasurementSelector.cpp b/Core/src/TrackFinding/MeasurementSelector.cpp index 71c8bab1d86..51d0064dbf1 100644 --- a/Core/src/TrackFinding/MeasurementSelector.cpp +++ b/Core/src/TrackFinding/MeasurementSelector.cpp @@ -15,6 +15,12 @@ namespace Acts { +MeasurementSelector::MeasurementSelector() + : m_config{{GeometryIdentifier(), MeasurementSelectorCuts{}}} {} + +MeasurementSelector::MeasurementSelector(const MeasurementSelectorCuts& cuts) + : m_config{{GeometryIdentifier(), cuts}} {} + MeasurementSelector::MeasurementSelector(Config config) : m_config(std::move(config)) {}