diff --git a/.github/workflows/checks.yml b/.github/workflows/checks.yml index 88d8d7a0ff7..655dc7af284 100644 --- a/.github/workflows/checks.yml +++ b/.github/workflows/checks.yml @@ -86,3 +86,15 @@ jobs: - name: Check run: > docs/parse_cmake_options.py CMakeLists.txt --write docs/getting_started.md --verify + spelling: + runs-on: ubuntu-latest + steps: + - uses: actions/checkout@v3 + - uses: actions/setup-python@v4 + - name: Install codespell + run: > + pip install codespell==2.2.5 + - name: Check + run: > + CI/check_spelling + diff --git a/CI/check_spelling b/CI/check_spelling new file mode 100755 index 00000000000..02910cfca0f --- /dev/null +++ b/CI/check_spelling @@ -0,0 +1,19 @@ +#!/bin/bash + +codespell -S "*.ipynb,*.onnx,_build,*.svg" -I ./CI/codespell_ignore.txt ./Core ./Examples ./Tests ./Plugins ./docs + +RETVAL=$? + +if [[ $RETVAL -ne 0 ]]; then + echo "" + echo "codespell detected incorrect spelling. You have two options how to deal with this:" + echo "" + echo "1) Apply one suggestion of codespell" + echo "2) If this is a false-positive, add the word to './CI/codespell_ignore.txt'" + echo " (Note that you need to add the word in lower case)" + echo "" + echo "You can apply suggestions interactively with" + echo "\`codespell -wi3 -I ./CI/codespell_ignore.txt \`" +fi + +exit $RETVAL diff --git a/CI/codespell_ignore.txt b/CI/codespell_ignore.txt new file mode 100644 index 00000000000..d9f7527d93e --- /dev/null +++ b/CI/codespell_ignore.txt @@ -0,0 +1,22 @@ +bu +strack +compres +coner +dthe +iself +sortings +fime +gaus +te +parm +writet +localy +lastr +digitial +exprot +pring +aline +boxs +ans +dthe +dthe diff --git a/Core/include/Acts/AmbiguityResolution/GreedyAmbiguityResolution.hpp b/Core/include/Acts/AmbiguityResolution/GreedyAmbiguityResolution.hpp index 0c15f333696..76490d76eae 100644 --- a/Core/include/Acts/AmbiguityResolution/GreedyAmbiguityResolution.hpp +++ b/Core/include/Acts/AmbiguityResolution/GreedyAmbiguityResolution.hpp @@ -42,7 +42,7 @@ class GreedyAmbiguityResolution { /// Maximum number of iterations std::uint32_t maximumIterations = 1000; - /// Minumum number of measurement to form a track. + /// Minimum number of measurement to form a track. size_t nMeasurementsMin = 7; }; @@ -75,7 +75,7 @@ class GreedyAmbiguityResolution { /// /// @param tracks The input track container. /// @param state An empty state object which is expected to be default constructed. - /// @param sourceLinkHash A functor to aquire a hash from a given source link. + /// @param sourceLinkHash A functor to acquire a hash from a given source link. /// @param sourceLinkEquality A functor to check equality of two source links. template class holder_t, typename source_link_hash_t, diff --git a/Core/include/Acts/AmbiguityResolution/GreedyAmbiguityResolution.ipp b/Core/include/Acts/AmbiguityResolution/GreedyAmbiguityResolution.ipp index 5aeed2de533..0be168f0b47 100644 --- a/Core/include/Acts/AmbiguityResolution/GreedyAmbiguityResolution.ipp +++ b/Core/include/Acts/AmbiguityResolution/GreedyAmbiguityResolution.ipp @@ -30,7 +30,7 @@ void GreedyAmbiguityResolution::computeInitialState( // count and chi2 and fill the measurement map in order to relate tracks to // each other if they have shared hits. for (const auto& track : tracks) { - // Kick out tracks that do not fullfil our initial requirements + // Kick out tracks that do not fulfill our initial requirements if (track.nMeasurements() < m_cfg.nMeasurementsMin) { continue; } diff --git a/Core/include/Acts/Clusterization/Clusterization.hpp b/Core/include/Acts/Clusterization/Clusterization.hpp index 1716d464a6d..e5971e696ca 100644 --- a/Core/include/Acts/Clusterization/Clusterization.hpp +++ b/Core/include/Acts/Clusterization/Clusterization.hpp @@ -84,7 +84,7 @@ template ClusterCollection mergeClusters(CellCollection& /*cells*/); /// @brief createClusters -/// Conveniance function which runs both labelClusters and createClusters. +/// Convenience function which runs both labelClusters and createClusters. template = 0 ? Value::Positive : Value::Negative; @@ -49,7 +49,7 @@ class Direction final { /// /// @param scalar is the signed value /// - /// @return a direciton enum + /// @return a direction enum static inline constexpr Direction fromScalarZeroAsPositive( ActsScalar scalar) { return scalar >= 0 ? Value::Positive : Value::Negative; diff --git a/Core/include/Acts/Definitions/Tolerance.hpp b/Core/include/Acts/Definitions/Tolerance.hpp index b3ed98ccb01..0380c732408 100644 --- a/Core/include/Acts/Definitions/Tolerance.hpp +++ b/Core/include/Acts/Definitions/Tolerance.hpp @@ -21,7 +21,7 @@ static constexpr ActsScalar s_epsilon = /// Tolerance for being on Surface /// /// @note This is intentionally given w/o an explicit unit to avoid having -/// to include the units header unneccessarily. With the native length +/// to include the units header unnecessarily. With the native length /// unit of mm this corresponds to 0.1um. static constexpr ActsScalar s_onSurfaceTolerance = 1e-4; diff --git a/Core/include/Acts/Definitions/TrackParametrization.hpp b/Core/include/Acts/Definitions/TrackParametrization.hpp index 8d22793f256..56c341fa11b 100644 --- a/Core/include/Acts/Definitions/TrackParametrization.hpp +++ b/Core/include/Acts/Definitions/TrackParametrization.hpp @@ -63,14 +63,14 @@ enum BoundIndices : unsigned int { /// in `namespace Acts` and are prefixed to avoid naming collisions. enum FreeIndices : unsigned int { // Spatial position - // The spatial position components must be stored as one continous block. + // The spatial position components must be stored as one continuous block. eFreePos0 = 0u, eFreePos1 = eFreePos0 + 1u, eFreePos2 = eFreePos0 + 2u, // Time eFreeTime = 3u, // (Unit) direction - // The direction components must be stored as one continous block. + // The direction components must be stored as one continuous block. eFreeDir0 = 4u, eFreeDir1 = eFreeDir0 + 1u, eFreeDir2 = eFreeDir0 + 2u, @@ -108,10 +108,10 @@ static_assert(FreeIndices::eFreeSize == 8u, static_assert(eBoundLoc0 != eBoundLoc1, "Local parameters must be different"); // Ensure free track parameter indices are consistently defined. -static_assert(eFreePos1 == eFreePos0 + 1u, "Position must be continous"); -static_assert(eFreePos2 == eFreePos0 + 2u, "Position must be continous"); -static_assert(eFreeDir1 == eFreeDir0 + 1u, "Direction must be continous"); -static_assert(eFreeDir2 == eFreeDir0 + 2u, "Direction must be continous"); +static_assert(eFreePos1 == eFreePos0 + 1u, "Position must be continuous"); +static_assert(eFreePos2 == eFreePos0 + 2u, "Position must be continuous"); +static_assert(eFreeDir1 == eFreeDir0 + 1u, "Direction must be continuous"); +static_assert(eFreeDir2 == eFreeDir0 + 2u, "Direction must be continuous"); // Shorthand vector/matrix types related to bound track parameters. using BoundVector = ActsVector; diff --git a/Core/include/Acts/Definitions/Units.hpp b/Core/include/Acts/Definitions/Units.hpp index 43215c3fea2..ff9c6a89543 100644 --- a/Core/include/Acts/Definitions/Units.hpp +++ b/Core/include/Acts/Definitions/Units.hpp @@ -124,7 +124,7 @@ namespace Acts { /// double bfield = 3.9_T; /// /// // convert output values (via unit constants) -/// doube t_in_ns = trackPars.time() / Acts::UnitConstants::ns; +/// double t_in_ns = trackPars.time() / Acts::UnitConstants::ns; /// // convert output values (via unit user literals) /// double x_in_mm = trackPars.position()[ePos0] / 1_mm; /// double p_in_TeV = trackPars.absoluteMomentum() / 1_TeV; diff --git a/Core/include/Acts/Detector/CylindricalContainerBuilder.hpp b/Core/include/Acts/Detector/CylindricalContainerBuilder.hpp index b92915a2ce5..e6fa47960c9 100644 --- a/Core/include/Acts/Detector/CylindricalContainerBuilder.hpp +++ b/Core/include/Acts/Detector/CylindricalContainerBuilder.hpp @@ -41,8 +41,8 @@ class CylindricalContainerBuilder : public IDetectorComponentBuilder { std::vector> builders = {}; /// Binning prescription of attachment std::vector binning = {}; - /// Auxilliary information, mainly for screen output - std::string auxilliary = ""; + /// Auxiliary information, mainly for screen output + std::string auxiliary = ""; }; /// Constructor with configuration arguments @@ -65,7 +65,7 @@ class CylindricalContainerBuilder : public IDetectorComponentBuilder { /// configuration object Config m_cfg; - /// Private acces method to the logger + /// Private access method to the logger const Logger& logger() const { return *m_logger; } /// logging instance diff --git a/Core/include/Acts/Detector/Detector.hpp b/Core/include/Acts/Detector/Detector.hpp index 4b351b66d50..6f868825bbf 100644 --- a/Core/include/Acts/Detector/Detector.hpp +++ b/Core/include/Acts/Detector/Detector.hpp @@ -32,7 +32,7 @@ class Detector : public std::enable_shared_from_this { /// /// @param name the detecor name /// @param rootVolumes the volumes contained by this detector - /// @param detectorVolumeUpdator is a Delegate to find the assocaited volume + /// @param detectorVolumeUpdator is a Delegate to find the associated volume /// /// @note will throw an exception if volumes vector is empty /// @note will throw an exception if duplicate volume names exist diff --git a/Core/include/Acts/Detector/DetectorBuilder.hpp b/Core/include/Acts/Detector/DetectorBuilder.hpp index 96dff53957f..6c3bc2c503c 100644 --- a/Core/include/Acts/Detector/DetectorBuilder.hpp +++ b/Core/include/Acts/Detector/DetectorBuilder.hpp @@ -33,8 +33,8 @@ class DetectorBuilder final : public IDetectorBuilder { std::string name = "unnamed"; /// An external builder std::shared_ptr builder = nullptr; - /// Auxilliary information - std::string auxilliary = ""; + /// Auxiliary information + std::string auxiliary = ""; }; /// Constructor with configuration arguments @@ -58,7 +58,7 @@ class DetectorBuilder final : public IDetectorBuilder { /// configuration object Config m_cfg; - /// Private acces method to the logger + /// Private access method to the logger const Logger& logger() const { return *m_logger; } /// logging instance diff --git a/Core/include/Acts/Detector/DetectorComponents.hpp b/Core/include/Acts/Detector/DetectorComponents.hpp index abf7afb6f2d..176e975b5f4 100644 --- a/Core/include/Acts/Detector/DetectorComponents.hpp +++ b/Core/include/Acts/Detector/DetectorComponents.hpp @@ -53,7 +53,7 @@ struct RootDetectorVolumes { struct DetectorComponent { /// Define a portal container using PortalContainer = std::map>; - /// The vector of construced volume(s) + /// The vector of constructed volume(s) std::vector> volumes = {}; /// The current map of outside portals PortalContainer portals = {}; diff --git a/Core/include/Acts/Detector/DetectorVolume.hpp b/Core/include/Acts/Detector/DetectorVolume.hpp index 52e5cb9b79c..db49f826a5d 100644 --- a/Core/include/Acts/Detector/DetectorVolume.hpp +++ b/Core/include/Acts/Detector/DetectorVolume.hpp @@ -52,7 +52,7 @@ class Detector; /// /// @note The construction of DetectorVolumes is done via a dedicated /// factory, this is necessary as then the shared_ptr is non-weak and it -/// can be registred in the portal generator for further geometry processing. +/// can be registered in the portal generator for further geometry processing. /// /// @note Navigation is always done by plain pointers, while /// object ownership is done by shared/unique pointers. @@ -98,8 +98,8 @@ class DetectorVolume : public std::enable_shared_from_this { /// @param transform the transform defining the volume position /// @param bounds the volume bounds /// @param surfaces are the contained surfaces of this volume - /// @param volumes are the containes volumes of this volume - /// @param detectorVolumeUpdator is a Delegate to find the assocaited volume + /// @param volumes are the contains volumes of this volume + /// @param detectorVolumeUpdator is a Delegate to find the associated volume /// @param surfaceCandidateUpdator the navigation state updator for surfaces/portals /// /// @note throws exception if misconfigured: no bounds @@ -173,7 +173,7 @@ class DetectorVolume : public std::enable_shared_from_this { /// Const access to the transform /// - /// @param gctx the geometry contect + /// @param gctx the geometry context /// /// @note the geometry context is currently ignored, but /// is a placeholder for eventually misaligned volumes @@ -184,7 +184,7 @@ class DetectorVolume : public std::enable_shared_from_this { /// Const access to the center /// - /// @param gctx the geometry contect + /// @param gctx the geometry context /// /// @note the geometry context is currently ignored, but /// is a placeholder for eventually misaligned volumes @@ -218,7 +218,7 @@ class DetectorVolume : public std::enable_shared_from_this { /// The Extent for this volume /// /// @param gctx is the geometry context - /// @param nseg is the number of segements to approximate + /// @param nseg is the number of segments to approximate /// /// @return an Extent object Extent extent(const GeometryContext& gctx, size_t nseg = 1) const; @@ -352,7 +352,7 @@ class DetectorVolume : public std::enable_shared_from_this { // Check containment - only in debug mode /// /// @param gctx the current geometry context object, e.g. alignment - /// @param nseg is the number of segements to approximate + /// @param nseg is the number of segments to approximate /// /// @return a boolean indicating if the objects are properly contained bool checkContainment(const GeometryContext& gctx, size_t nseg = 1) const; diff --git a/Core/include/Acts/Detector/DetectorVolumeBuilder.hpp b/Core/include/Acts/Detector/DetectorVolumeBuilder.hpp index 9d9c83f2ade..0cf3f0bc252 100644 --- a/Core/include/Acts/Detector/DetectorVolumeBuilder.hpp +++ b/Core/include/Acts/Detector/DetectorVolumeBuilder.hpp @@ -43,8 +43,8 @@ class DetectorVolumeBuilder : public IDetectorComponentBuilder { std::shared_ptr internalsBuilder = nullptr; /// Add eventual internal volume to root bool addInternalsToRoot = false; - /// Auxilliary information - std::string auxilliary = ""; + /// Auxiliary information + std::string auxiliary = ""; }; /// Constructor with configuration arguments @@ -68,7 +68,7 @@ class DetectorVolumeBuilder : public IDetectorComponentBuilder { /// configuration object Config m_cfg; - /// Private acces method to the logger + /// Private access method to the logger const Logger& logger() const { return *m_logger; } /// logging instance diff --git a/Core/include/Acts/Detector/KdtSurfacesProvider.hpp b/Core/include/Acts/Detector/KdtSurfacesProvider.hpp index 99c7752b580..b2120a6f17f 100644 --- a/Core/include/Acts/Detector/KdtSurfacesProvider.hpp +++ b/Core/include/Acts/Detector/KdtSurfacesProvider.hpp @@ -111,7 +111,7 @@ class KdtSurfaces { /// Cast values that turn a global position to lookup position std::array m_casts = {}; - /// Helper to generate refernce points for filling + /// Helper to generate reference points for filling reference_generator m_rGenerator; /// Unroll the cast loop diff --git a/Core/include/Acts/Detector/LayerStructureBuilder.hpp b/Core/include/Acts/Detector/LayerStructureBuilder.hpp index 2f065a7f1cc..a91fc372e79 100644 --- a/Core/include/Acts/Detector/LayerStructureBuilder.hpp +++ b/Core/include/Acts/Detector/LayerStructureBuilder.hpp @@ -72,7 +72,7 @@ class LayerStructureBuilder : public IInternalStructureBuilder { std::vector> m_surfaces = {}; }; - /// @brief Support parameter defintions + /// @brief Support parameter definitions struct Support { /// Define whether you want to build support structures std::array values = {}; @@ -102,7 +102,7 @@ class LayerStructureBuilder : public IInternalStructureBuilder { /// Polyhedron approximations unsigned int nSegments = 1u; /// Extra information, mainly for screen output - std::string auxilliary = ""; + std::string auxiliary = ""; }; /// Constructor @@ -124,7 +124,7 @@ class LayerStructureBuilder : public IInternalStructureBuilder { /// configuration object Config m_cfg; - /// Private acces method to the logger + /// Private access method to the logger const Logger& logger() const { return *m_logger; } /// logging instance diff --git a/Core/include/Acts/Detector/PortalGenerators.hpp b/Core/include/Acts/Detector/PortalGenerators.hpp index 5416233670f..d8f8bf50614 100644 --- a/Core/include/Acts/Detector/PortalGenerators.hpp +++ b/Core/include/Acts/Detector/PortalGenerators.hpp @@ -23,7 +23,7 @@ namespace Experimental { class DetectorVolume; class Portal; -/// The Portal genertor definition +/// The Portal generator definition using PortalGenerator = Delegate>( const Transform3&, const VolumeBounds&, const std::shared_ptr&)>; @@ -47,7 +47,7 @@ PortalGenerator defaultPortalGenerator(); /// @brief Calls the portal generation and adds registration to sub portals /// /// This code is split off the PortalGenerator code in order to allow -/// unit testing of the portal generation wihtout detector volume construction +/// unit testing of the portal generation without detector volume construction /// /// @param dTransform a contextually resolved transform /// @param dBounds the detecor volume bounds diff --git a/Core/include/Acts/Detector/VolumeStructureBuilder.hpp b/Core/include/Acts/Detector/VolumeStructureBuilder.hpp index 9b02435e77a..ca2acb79b78 100644 --- a/Core/include/Acts/Detector/VolumeStructureBuilder.hpp +++ b/Core/include/Acts/Detector/VolumeStructureBuilder.hpp @@ -46,8 +46,8 @@ class VolumeStructureBuilder : public IExternalStructureBuilder { std::vector boundValues = {}; /// The optional extent to feed into the values std::optional extent = std::nullopt; - /// Some auxilliary information - std::string auxilliary = ""; + /// Some auxiliary information + std::string auxiliary = ""; }; /// Constructor diff --git a/Core/include/Acts/Detector/detail/GridAxisGenerators.hpp b/Core/include/Acts/Detector/detail/GridAxisGenerators.hpp index f14c28fb909..dc7d79d4d7a 100644 --- a/Core/include/Acts/Detector/detail/GridAxisGenerators.hpp +++ b/Core/include/Acts/Detector/detail/GridAxisGenerators.hpp @@ -24,7 +24,7 @@ namespace detail { /// /// The call operator() API allows to plug axis generators into /// dedicated code snippets and create fitting axis types on the fly -/// which then turn into conrete Grid types. +/// which then turn into concrete Grid types. /// namespace GridAxisGenerators { @@ -58,7 +58,7 @@ using EqBound = Eq; using EqOpen = Eq; using EqClosed = Eq; -/// @brief Templated base generator for vairable axis as a tuple - 1D +/// @brief Templated base generator for variable axis as a tuple - 1D /// /// @tparam aType the type of the axis (Bound, Closed, Open) template diff --git a/Core/include/Acts/Detector/detail/IndexedGridFiller.hpp b/Core/include/Acts/Detector/detail/IndexedGridFiller.hpp index 8b95a9142e0..e640737f476 100644 --- a/Core/include/Acts/Detector/detail/IndexedGridFiller.hpp +++ b/Core/include/Acts/Detector/detail/IndexedGridFiller.hpp @@ -193,7 +193,7 @@ std::set localIndices( /// /// @param lbins the local bins /// -/// @return a string containing the local bins orderd in a set +/// @return a string containing the local bins ordered in a set template std::string outputIndices(const std::set& lbins) { std::string rString; @@ -259,7 +259,7 @@ struct IndexedGridFiller { std::vector gridQueries; gridQueries.reserve(refs.size()); for (const auto& ref : refs) { - // Cast the transfrom according to the grid binning + // Cast the transform according to the grid binning gridQueries.push_back(iGrid.castPosition(ref)); } ACTS_DEBUG(gridQueries.size() << " reference points generated."); diff --git a/Core/include/Acts/Detector/detail/IndexedSurfacesGenerator.hpp b/Core/include/Acts/Detector/detail/IndexedSurfacesGenerator.hpp index 9a501a17025..7ae7efe0431 100644 --- a/Core/include/Acts/Detector/detail/IndexedSurfacesGenerator.hpp +++ b/Core/include/Acts/Detector/detail/IndexedSurfacesGenerator.hpp @@ -79,7 +79,7 @@ struct IndexedSurfacesGenerator { IndexedSurfacesImpl indexedSurfaces(std::move(grid), bvArray, transform); - // Fill the bin indicies + // Fill the bin indices IndexedGridFiller filler{binExpansion}; filler.oLogger = oLogger->cloneWithSuffix("_filler"); filler.fill(gctx, indexedSurfaces, surfaces, rGenerator, assignToAll); diff --git a/Core/include/Acts/Detector/detail/ReferenceGenerators.hpp b/Core/include/Acts/Detector/detail/ReferenceGenerators.hpp index cd0c78add4f..e0e087db1af 100644 --- a/Core/include/Acts/Detector/detail/ReferenceGenerators.hpp +++ b/Core/include/Acts/Detector/detail/ReferenceGenerators.hpp @@ -30,14 +30,14 @@ struct CenterReferenceGenerator { /// @param gctx the geometry context of this operation /// @param surface the surface for which the reference point is to be accessed /// - /// @return a vector of referene points for filling + /// @return a vector of reference points for filling const std::vector references(const GeometryContext& gctx, const Surface& surface) const { return {surface.center(gctx)}; } }; -/// A struct to access reference postions based on bin values +/// A struct to access reference positions based on bin values /// /// This generator will provide only one filling point and hence /// only a single bin in the indexed grid. @@ -45,12 +45,12 @@ struct BinningValueReferenceGenerator { /// The binning value BinningValue bValue = BinningValue::binValues; - /// Helper to access a reference postion based on binning value + /// Helper to access a reference position based on binning value /// /// @param gctx the geometry context of this operation /// @param surface the surface for which the reference point is to be accessed /// - /// @return a vector of referene points for filling + /// @return a vector of reference points for filling const std::vector references(const GeometryContext& gctx, const Surface& surface) const { return {surface.binningPosition(gctx, bValue)}; @@ -75,7 +75,7 @@ struct PolyhedronReferenceGenerator { /// @param gctx the geometry context of this operation /// @param surface the surface for which the reference point is to be accessed /// - /// @return a vector of referene points for filling + /// @return a vector of reference points for filling const std::vector references(const GeometryContext& gctx, const Surface& surface) const { // Create the return vector diff --git a/Core/include/Acts/Detector/detail/SupportHelper.hpp b/Core/include/Acts/Detector/detail/SupportHelper.hpp index d2f413d31f2..70f8ef895d6 100644 --- a/Core/include/Acts/Detector/detail/SupportHelper.hpp +++ b/Core/include/Acts/Detector/detail/SupportHelper.hpp @@ -56,7 +56,7 @@ std::vector> discSupport( /// @param assignToAll [in, out] indices that are assigned to all bins in the indexing /// @param layerExtent the externally provided layer Extent /// @param layerRepresentation the shape of the layer -/// @param layerSupportValues the support structue in numbers +/// @param layerSupportValues the support structure in numbers /// @param layerTransform is an optional value of the layer transform /// @param supportSplits the number of splits if splitting is configured /// diff --git a/Core/include/Acts/Digitization/CartesianSegmentation.hpp b/Core/include/Acts/Digitization/CartesianSegmentation.hpp index 4f594aa4807..a509240642a 100644 --- a/Core/include/Acts/Digitization/CartesianSegmentation.hpp +++ b/Core/include/Acts/Digitization/CartesianSegmentation.hpp @@ -24,7 +24,7 @@ namespace Acts { /// @brief Segmentation Base class /// -/// Segementation class for generic pixel, strixels and strip segmentations +/// Segmentation class for generic pixel, strixels and strip segmentations /// in a cartesian frame, this uses a cartesian X/Y local surface definition /// /// The calculation can be done in full 3D, i.e. the segments of the path @@ -32,7 +32,7 @@ namespace Acts { /// the planar module are calculated in a 3D model - or in 2D, when the entire /// calculation is done on the projective surface. When the 2D option is used, /// segmentation surfaces are not created. The 2D calculation is faster and uses -/// less memory, however, effects witin the sensor volume can not be easily +/// less memory, however, effects within the sensor volume can not be easily /// integrated /// /// Conventions: @@ -60,7 +60,7 @@ class CartesianSegmentation : public Segmentation { /// optimisation /// /// @note if both RectangleBounds and BinUtility are provided, no check is - /// done for consitency + /// done for consistency CartesianSegmentation(std::shared_ptr bUtility, std::shared_ptr mBounds = nullptr); diff --git a/Core/include/Acts/Digitization/DigitizationModule.hpp b/Core/include/Acts/Digitization/DigitizationModule.hpp index b67c7ef4fdb..57586e77b2f 100644 --- a/Core/include/Acts/Digitization/DigitizationModule.hpp +++ b/Core/include/Acts/Digitization/DigitizationModule.hpp @@ -115,7 +115,7 @@ class DigitizationModule { DigitizationStep digitizationStep(const Vector3& start, const Vector3& end) const; - /// Return the bounding surfaces inlcuding top and bottom + /// Return the bounding surfaces including top and bottom const SurfacePtrVector& boundarySurfaces() const; /// Return all surfaces in X - excluding the boundaries diff --git a/Core/include/Acts/Digitization/PlanarModuleStepper.hpp b/Core/include/Acts/Digitization/PlanarModuleStepper.hpp index 4f5eebf5c59..b0177b22314 100644 --- a/Core/include/Acts/Digitization/PlanarModuleStepper.hpp +++ b/Core/include/Acts/Digitization/PlanarModuleStepper.hpp @@ -32,7 +32,7 @@ class PlanarModuleStepper { public: /// Constructor /// - /// @param mlogger is the logging istance + /// @param mlogger is the logging instance PlanarModuleStepper(std::unique_ptr mlogger = getDefaultLogger( "PlanarModuleStepper", Logging::INFO)); @@ -44,7 +44,7 @@ class PlanarModuleStepper { /// @param gctx The current geometry context object, e.g. alignment /// @param dmodule is the digitization module /// @param startPoint is the starting position of the stepping - /// @param endPoint is the end postion of the stepping + /// @param endPoint is the end position of the stepping /// /// @return is the vector of digitization steps std::vector cellSteps(const GeometryContext& gctx, @@ -57,7 +57,7 @@ class PlanarModuleStepper { /// @param gctx The current geometry context object, e.g. alignment /// @param dmodule is the digitization module /// @param moduleIntersection is the 2d intersection at the module surface - /// @param trackDirection is the track direction at the instersection + /// @param trackDirection is the track direction at the intersection /// /// @return is the vector of digitization steps std::vector cellSteps(const GeometryContext& gctx, diff --git a/Core/include/Acts/Digitization/Segmentation.hpp b/Core/include/Acts/Digitization/Segmentation.hpp index b0070e970aa..702d94c33c0 100644 --- a/Core/include/Acts/Digitization/Segmentation.hpp +++ b/Core/include/Acts/Digitization/Segmentation.hpp @@ -57,7 +57,7 @@ class Segmentation { SurfacePtrVector& segmentationSurfacesY, double halfThickness, int readoutDirection, double lorentzAngle) const = 0; - /// Get the digitization cell fropm a 3D position + /// Get the digitization cell from a 3D position /// - ignores the shift, i.e. assumenes in to be in cell frame /// /// @param position is the position for which the cell is requested @@ -65,7 +65,7 @@ class Segmentation { /// @return is a cell with cell ids virtual DigitizationCell cell(const Vector3& position) const = 0; - /// Get the digitization cell fropm a 2D position + /// Get the digitization cell from a 2D position /// - ignores the shift, i.e. assumenes in to be in cell frame /// /// @param position is the position for which the cell is requested diff --git a/Core/include/Acts/EventData/Charge.hpp b/Core/include/Acts/EventData/Charge.hpp index 6ea7a652fbc..e6df928ffe1 100644 --- a/Core/include/Acts/EventData/Charge.hpp +++ b/Core/include/Acts/EventData/Charge.hpp @@ -122,7 +122,7 @@ struct SinglyCharged { /// approximate comparison with an arbitrary epsilon. class AnyCharge { public: - /// Delete default constructor to ensure charge is always explicitely given. + /// Delete default constructor to ensure charge is always explicitly given. AnyCharge() = delete; /// Construct with the magnitude of the input charge. template diff --git a/Core/include/Acts/EventData/Measurement.hpp b/Core/include/Acts/EventData/Measurement.hpp index 28bde195c03..8c13b204614 100644 --- a/Core/include/Acts/EventData/Measurement.hpp +++ b/Core/include/Acts/EventData/Measurement.hpp @@ -126,10 +126,10 @@ class Measurement { /// Expansion matrix from the measured subspace into the full space. /// - /// This is equivalent to the tranpose of the projection matrix only in the + /// This is equivalent to the transpose of the projection matrix only in the /// case of a trivial projection matrix. While this is the case here, it is /// still recommended to use the expansion matrix directly in cases where it - /// is explicitely used. + /// is explicitly used. ExpansionMatrix expander() const { return m_subspace.template expander(); } diff --git a/Core/include/Acts/EventData/MultiComponentBoundTrackParameters.hpp b/Core/include/Acts/EventData/MultiComponentBoundTrackParameters.hpp index cbacafd17d7..f1b292911c2 100644 --- a/Core/include/Acts/EventData/MultiComponentBoundTrackParameters.hpp +++ b/Core/include/Acts/EventData/MultiComponentBoundTrackParameters.hpp @@ -18,7 +18,7 @@ namespace Acts { -/// This class is only a light wrapper arround a surface and a vector of +/// This class is only a light wrapper around a surface and a vector of /// parameters. Its main purpose is to provide many constructors for the /// underlying vector. Most accessors are generated from the /// SingleBoundTrackParameters equivalent and thus may be expensive @@ -110,7 +110,7 @@ class MultiComponentBoundTrackParameters { /// @param cov Bound parameters covariance matrix /// /// In principle, only the charge magnitude is needed her to allow - /// unambigous extraction of the absolute momentum. The particle charge is + /// unambiguous extraction of the absolute momentum. The particle charge is /// required as an input here to be consistent with the other constructors /// below that that also take the charge as an input. The charge sign is /// only used in debug builds to check for consistency with the q/p diff --git a/Core/include/Acts/EventData/MultiTrajectory.hpp b/Core/include/Acts/EventData/MultiTrajectory.hpp index 3f2916420c1..8d96ebb5b35 100644 --- a/Core/include/Acts/EventData/MultiTrajectory.hpp +++ b/Core/include/Acts/EventData/MultiTrajectory.hpp @@ -315,7 +315,7 @@ class TrackStateProxy { return component(); } - /// Return whather this track state has a previous (parent) track state. + /// Return whether this track state has a previous (parent) track state. /// @return Boolean indicating whether a previous track state exists bool hasPrevious() const { return component() != kInvalid; @@ -352,7 +352,7 @@ class TrackStateProxy { shareFrom(other, component, component); } - /// Share a shareable component from anothe track state + /// Share a shareable component from another track state /// @param shareSource Which component to share from /// @param shareTarget Which component to share as. This can be be different from /// as @p shareSource, e.g. predicted can be shared as filtered. diff --git a/Core/include/Acts/EventData/SingleBoundTrackParameters.hpp b/Core/include/Acts/EventData/SingleBoundTrackParameters.hpp index 65ad1c9363c..da9586579ea 100644 --- a/Core/include/Acts/EventData/SingleBoundTrackParameters.hpp +++ b/Core/include/Acts/EventData/SingleBoundTrackParameters.hpp @@ -46,7 +46,7 @@ class SingleBoundTrackParameters { /// @param q Particle charge /// @param cov Bound parameters covariance matrix /// - /// In principle, only the charge magnitude is needed her to allow unambigous + /// In principle, only the charge magnitude is needed her to allow unambiguous /// extraction of the absolute momentum. The particle charge is required as /// an input here to be consistent with the other constructors below that /// that also take the charge as an input. The charge sign is only used in @@ -160,7 +160,7 @@ class SingleBoundTrackParameters { /// Optional covariance matrix. const std::optional& covariance() const { return m_cov; } - /// Access a single parameter value indentified by its index. + /// Access a single parameter value identified by its index. /// /// @tparam kIndex Track parameter index template @@ -258,7 +258,7 @@ class SingleBoundTrackParameters { /// @note Comparing track parameters for bitwise equality is not a good /// idea. /// Depending on the context you might want to compare only the - /// parameter values, or compare them for compability instead of + /// parameter values, or compare them for compatibility instead of /// equality; you might also have different (floating point) thresholds /// of equality in different contexts. None of that can be handled by /// this operator. Users should think really hard if this is what they diff --git a/Core/include/Acts/EventData/SingleFreeTrackParameters.hpp b/Core/include/Acts/EventData/SingleFreeTrackParameters.hpp index 76aa99232a2..f466a07143b 100644 --- a/Core/include/Acts/EventData/SingleFreeTrackParameters.hpp +++ b/Core/include/Acts/EventData/SingleFreeTrackParameters.hpp @@ -38,7 +38,7 @@ class SingleFreeTrackParameters { /// @param q Particle charge /// @param cov Free parameters covariance matrix /// - /// In principle, only the charge magnitude is needed her to allow unambigous + /// In principle, only the charge magnitude is needed her to allow unambiguous /// extraction of the absolute momentum. The particle charge is required as /// an input here to be consistent with the other constructors below that /// that also take the charge as an input. The charge sign is only used in @@ -133,7 +133,7 @@ class SingleFreeTrackParameters { /// Optional covariance matrix. const std::optional& covariance() const { return m_cov; } - /// Access a single parameter value indentified by its index. + /// Access a single parameter value identified by its index. /// /// @tparam kIndex Track parameter index template diff --git a/Core/include/Acts/EventData/TrackContainer.hpp b/Core/include/Acts/EventData/TrackContainer.hpp index d0134be438f..18f5a724f0e 100644 --- a/Core/include/Acts/EventData/TrackContainer.hpp +++ b/Core/include/Acts/EventData/TrackContainer.hpp @@ -31,7 +31,7 @@ template struct IsReadOnlyTrackContainer; /// Track container interface class. This type represents a collections of -/// tracks. It uses a backend to store bothe the actual tracks and the +/// tracks. It uses a backend to store both the actual tracks and the /// associated track states. /// @tparam track_container_t the track container backend /// @tparam traj_t the track state container backend @@ -130,7 +130,7 @@ class TrackContainer { /// Remove a track at index @p itrack from the container /// @note This invalidates all track proxies! - /// @param itrack The index of the track to remmove + /// @param itrack The index of the track to remove template > void removeTrack(IndexType itrack) { m_container->removeTrack_impl(itrack); diff --git a/Core/include/Acts/EventData/detail/CorrectedTransformationFreeToBound.hpp b/Core/include/Acts/EventData/detail/CorrectedTransformationFreeToBound.hpp index 964f005faf5..bc8c699525e 100644 --- a/Core/include/Acts/EventData/detail/CorrectedTransformationFreeToBound.hpp +++ b/Core/include/Acts/EventData/detail/CorrectedTransformationFreeToBound.hpp @@ -40,14 +40,14 @@ struct FreeToBoundCorrection { /// Construct from boolean and UKF parameters (alpha, beta) /// - /// @param apply_ Wheter to apply correction + /// @param apply_ Whether to apply correction /// @param alpha_ The UKF tuning parameter alpha /// @param beta_ The UKF tuning parameter beta FreeToBoundCorrection(bool apply_, ActsScalar alpha_, ActsScalar beta_); /// Construct from boolean only /// - /// @param apply_ Wheter to apply correction + /// @param apply_ Whether to apply correction explicit FreeToBoundCorrection(bool apply_); /// Return boolean for applying correction or not diff --git a/Core/include/Acts/EventData/detail/ParameterTraits.hpp b/Core/include/Acts/EventData/detail/ParameterTraits.hpp index f68f21e57f0..5e3b0df0163 100644 --- a/Core/include/Acts/EventData/detail/ParameterTraits.hpp +++ b/Core/include/Acts/EventData/detail/ParameterTraits.hpp @@ -105,7 +105,7 @@ struct CyclicParameterTraits { // Limit types for parameter traits. // -// The functions names are chosen to be consisten w/ std::numeric_limits +// The functions names are chosen to be consistent w/ std::numeric_limits struct PhiBoundParameterLimits { static constexpr double lowest() { return -M_PI; } static constexpr double max() { return M_PI; } @@ -131,7 +131,7 @@ struct ParameterTraitsImpl { }; template struct ParameterTraitsImpl { - // other bound parameters not explicitely specified above are unrestricted + // other bound parameters not explicitly specified above are unrestricted using Type = UnrestrictedParameterTraits; }; template diff --git a/Core/include/Acts/Geometry/AbstractVolume.hpp b/Core/include/Acts/Geometry/AbstractVolume.hpp index c7e88216932..4375e31448f 100644 --- a/Core/include/Acts/Geometry/AbstractVolume.hpp +++ b/Core/include/Acts/Geometry/AbstractVolume.hpp @@ -43,14 +43,14 @@ using VolumeBoundsPtr = std::shared_ptr; /// the Acts::VolumeBounds that carry a decompose method. /// /// Boundary surfaces can be shared between AbstractVolumes to enhance automatic -/// navigation between AbstractVolumes, therefor they are reference counted by a -/// std::shared_ptr holder class. +/// navigation between AbstractVolumes, therefore they are reference counted by +/// a std::shared_ptr holder class. class AbstractVolume : public Volume { public: /// Constructor with shared Transform3*, VolumeBounds* /// - /// @param transform is the gobal 3d transformation into the volume frame + /// @param transform is the global 3d transformation into the volume frame /// @param volbounds is the boundary definition AbstractVolume(const Transform3& transform, VolumeBoundsPtr volbounds); diff --git a/Core/include/Acts/Geometry/BoundarySurfaceFace.hpp b/Core/include/Acts/Geometry/BoundarySurfaceFace.hpp index 92dfc95efb9..2575619aa2a 100644 --- a/Core/include/Acts/Geometry/BoundarySurfaceFace.hpp +++ b/Core/include/Acts/Geometry/BoundarySurfaceFace.hpp @@ -16,13 +16,13 @@ namespace Acts { /// /// Enum to describe the position of the BoundarySurface /// respectively to the frame orientatin of the volume, -/// this is mainly ment for code readability. +/// this is mainly meant for code readability. /// /// The different numeration sequences can be found in the /// documentation of the actual VolumeBounds implementations. /// /// The order of faces is chosen to follow - as much as -/// possible - a cycular structure. +/// possible - a circular structure. enum BoundarySurfaceFace { negativeFaceXY = 0, positiveFaceXY = 1, diff --git a/Core/include/Acts/Geometry/BoundarySurfaceT.hpp b/Core/include/Acts/Geometry/BoundarySurfaceT.hpp index 29f839e5cb2..84367785852 100644 --- a/Core/include/Acts/Geometry/BoundarySurfaceT.hpp +++ b/Core/include/Acts/Geometry/BoundarySurfaceT.hpp @@ -54,8 +54,8 @@ class BoundarySurfaceT { /// Constructor for a Boundary with exact two Volumes attached to it /// - usually used in a volume constructor /// - /// @param surface The unqiue surface the boundary represents - /// @param inside The inside volume the bounday surface points to + /// @param surface The unique surface the boundary represents + /// @param inside The inside volume the boundary surface points to /// @param outside The outside volume the boundary surface points to BoundarySurfaceT(std::shared_ptr surface, const volume_t* inside, const volume_t* outside) @@ -68,8 +68,8 @@ class BoundarySurfaceT { /// Constructor for a Boundary with exact two Volumes attached to it /// - usually used in a volume constructor /// - /// @param surface The unqiue surface the boundary represents - /// @param inside The inside volume the bounday surface points to + /// @param surface The unique surface the boundary represents + /// @param inside The inside volume the boundary surface points to /// @param outside The outside volume the boundary surface points to BoundarySurfaceT(std::shared_ptr surface, VolumePtr inside, VolumePtr outside) @@ -82,8 +82,8 @@ class BoundarySurfaceT { /// Constructor for a Boundary with exact multiple Volumes attached to it /// - usually used in a volume constructor /// - /// @param surface The unqiue surface the boundary represents - /// @param insideArray The inside volume array the bounday surface points to + /// @param surface The unique surface the boundary represents + /// @param insideArray The inside volume array the boundary surface points to /// @param outsideArray The outside volume array the boundary surface /// points to BoundarySurfaceT(std::shared_ptr surface, @@ -103,7 +103,7 @@ class BoundarySurfaceT { /// @param gctx The current geometry context object, e.g. alignment /// @param pos The global position on surface /// @param mom The direction on the surface - /// @param dir is an aditional direction corrective + /// @param dir is an additional direction corrective /// /// @return The attached volume at that position virtual const volume_t* attachedVolume(const GeometryContext& gctx, diff --git a/Core/include/Acts/Geometry/ConeLayer.hpp b/Core/include/Acts/Geometry/ConeLayer.hpp index 22f89b7b507..a9a6346bb17 100644 --- a/Core/include/Acts/Geometry/ConeLayer.hpp +++ b/Core/include/Acts/Geometry/ConeLayer.hpp @@ -30,14 +30,14 @@ class ConeLayer : virtual public ConeSurface, public Layer { public: /// Factory for shared layer /// - /// @param transform is the 3D transform that poisitions the layer in 3D frame + /// @param transform is the 3D transform that positions the layer in 3D frame /// @param cbounds is the conical bound description /// @param surfaceArray is the array of sensitive surfaces /// @param thickness is the layer thickness along the normal axis /// @param ad is the approach descriptor for navigation towards the layer /// @param laytyp is the layer type /// - /// @todo chage od and ad to unique_ptr + /// @todo change od and ad to unique_ptr /// /// @return is a shared pointer to a layer static MutableLayerPtr create( @@ -64,14 +64,14 @@ class ConeLayer : virtual public ConeSurface, public Layer { protected: /// Private constructor with arguments /// - /// @param transform is the 3D transform that poisitions the layer in 3D frame + /// @param transform is the 3D transform that positions the layer in 3D frame /// @param cbounds is the conical bound description /// @param surfaceArray is the array of sensitive surfaces /// @param thickness is the layer thickness along the normal axis /// @param ade is the approach descriptor for navigation towards the layer /// @param laytyp is the layer type /// - /// @todo chage od and ad to unique_ptr + /// @todo change od and ad to unique_ptr ConeLayer(const Transform3& transform, std::shared_ptr cbounds, std::unique_ptr surfaceArray, double thickness = 0., diff --git a/Core/include/Acts/Geometry/ConeVolumeBounds.hpp b/Core/include/Acts/Geometry/ConeVolumeBounds.hpp index 771fc0b07f8..93b7283e9ae 100644 --- a/Core/include/Acts/Geometry/ConeVolumeBounds.hpp +++ b/Core/include/Acts/Geometry/ConeVolumeBounds.hpp @@ -159,7 +159,7 @@ class ConeVolumeBounds : public VolumeBounds { /// Create the surface bounds void buildSurfaceBounds(); - /// Templated dump methos + /// Templated dump methods /// @tparam stream_t The type of the stream for dumping /// @param dt The stream object template diff --git a/Core/include/Acts/Geometry/CuboidVolumeBounds.hpp b/Core/include/Acts/Geometry/CuboidVolumeBounds.hpp index 4ccca9d70c8..ae393b3d64d 100644 --- a/Core/include/Acts/Geometry/CuboidVolumeBounds.hpp +++ b/Core/include/Acts/Geometry/CuboidVolumeBounds.hpp @@ -133,7 +133,7 @@ class CuboidVolumeBounds : public VolumeBounds { private: /// Templated dumpT method - /// @tparam stream_t The type fo the dump stream + /// @tparam stream_t The type for the dump stream /// @param dt The dump stream object template stream_t& dumpT(stream_t& dt) const; diff --git a/Core/include/Acts/Geometry/CuboidVolumeBuilder.hpp b/Core/include/Acts/Geometry/CuboidVolumeBuilder.hpp index aa9a3a776e3..17e1d45d1bc 100644 --- a/Core/include/Acts/Geometry/CuboidVolumeBuilder.hpp +++ b/Core/include/Acts/Geometry/CuboidVolumeBuilder.hpp @@ -79,7 +79,7 @@ class CuboidVolumeBuilder : public ITrackingVolumeBuilder { std::array envelopeY{0, 0}; // Envelope in Z std::array envelopeZ{0, 0}; - // An optional rotation fo this + // An optional rotation for this std::optional rotation{std::nullopt}; }; @@ -137,7 +137,7 @@ class CuboidVolumeBuilder : public ITrackingVolumeBuilder { std::shared_ptr buildSurface(const GeometryContext& gctx, const SurfaceConfig& cfg) const; - /// @brief This function creates a layer with a surface encaspulated with a + /// @brief This function creates a layer with a surface encapsulated with a /// given configuration. The surface gets a detector element attached if the /// template parameter is non-void. /// diff --git a/Core/include/Acts/Geometry/CutoutCylinderVolumeBounds.hpp b/Core/include/Acts/Geometry/CutoutCylinderVolumeBounds.hpp index 99c6e52af56..807e918e120 100644 --- a/Core/include/Acts/Geometry/CutoutCylinderVolumeBounds.hpp +++ b/Core/include/Acts/Geometry/CutoutCylinderVolumeBounds.hpp @@ -23,7 +23,7 @@ namespace Acts { class CylinderBounds; class DiscBounds; -/// Class which implements a cutout cylinder. This shape is bascially a +/// Class which implements a cutout cylinder. This shape is basically a /// cylinder, with another, smaller cylinder subtracted from the center. /// --------------------- rmax /// | | diff --git a/Core/include/Acts/Geometry/CylinderLayer.hpp b/Core/include/Acts/Geometry/CylinderLayer.hpp index e72784d6590..2493a31f785 100644 --- a/Core/include/Acts/Geometry/CylinderLayer.hpp +++ b/Core/include/Acts/Geometry/CylinderLayer.hpp @@ -41,7 +41,7 @@ class CylinderLayer : public CylinderSurface, public Layer { /// /// @todo ApproachDescriptor to unique_ptr /// - /// @return The return object is a shared poiter to the layer. + /// @return The return object is a shared pointer to the layer. static std::shared_ptr create( const Transform3& transform, const std::shared_ptr& cbounds, diff --git a/Core/include/Acts/Geometry/CylinderVolumeBounds.hpp b/Core/include/Acts/Geometry/CylinderVolumeBounds.hpp index 0f82917c697..6d12db56caf 100644 --- a/Core/include/Acts/Geometry/CylinderVolumeBounds.hpp +++ b/Core/include/Acts/Geometry/CylinderVolumeBounds.hpp @@ -209,7 +209,7 @@ class CylinderVolumeBounds : public VolumeBounds { void buildSurfaceBounds(); /// Templated dumpT method - /// @tparam stream_t The type fo the dump stream + /// @tparam stream_t The type for the dump stream /// @param dt The dump stream object template stream_t& dumpT(stream_t& dt) const; diff --git a/Core/include/Acts/Geometry/CylinderVolumeBuilder.hpp b/Core/include/Acts/Geometry/CylinderVolumeBuilder.hpp index cadfffd43c2..73d45ed63be 100644 --- a/Core/include/Acts/Geometry/CylinderVolumeBuilder.hpp +++ b/Core/include/Acts/Geometry/CylinderVolumeBuilder.hpp @@ -252,7 +252,7 @@ struct WrappingConfig { cVolumeConfig.midPointAttachZ(pVolumeConfig); } // adapt r afterwards - // - easy if no exisitng volume + // - easy if no existing volume // - possible if no central volume if (!existingVolumeConfig || !cVolumeConfig) { nVolumeConfig.adaptR(containerVolumeConfig); @@ -273,13 +273,13 @@ struct WrappingConfig { nVolumeConfig.attachZ(existingVolumeConfig); // will attach the new volume(s) wCondition = Attaching; - wConditionScreen = "[n attched]"; + wConditionScreen = "[n attached]"; } if (pVolumeConfig && pVolumeConfig.zMin > existingVolumeConfig.zMax) { pVolumeConfig.attachZ(existingVolumeConfig); // will attach the new volume(s) wCondition = Attaching; - wConditionScreen = "[p attched]"; + wConditionScreen = "[p attached]"; } // see if inner glue volumes are needed if (containerVolumeConfig.rMin > existingVolumeConfig.rMin) { @@ -418,7 +418,7 @@ struct WrappingConfig { sl << "New container built with configuration: " << containerVolumeConfig.toString() << '\n'; } - // go throug the new new ones first + // go through the new new ones first if (nVolumeConfig) { sl << " - n: Negative Endcap, current configuration: " << nVolumeConfig.toString() << '\n'; @@ -474,7 +474,7 @@ class CylinderVolumeBuilder : public ITrackingVolumeBuilder { /// @struct Config /// Nested configuration struct for this CylinderVolumeBuilder struct Config { - /// The trackign volume helper for construction + /// The tracking volume helper for construction std::shared_ptr trackingVolumeHelper = nullptr; /// The string based indenfication std::string volumeName = ""; @@ -497,7 +497,7 @@ class CylinderVolumeBuilder : public ITrackingVolumeBuilder { /// the additional envelope in Z to create zMin, zMax double layerEnvelopeZ = 1. * UnitConstants::mm; - // The potential boundary material (MB) options - there are 6 at maximium + // The potential boundary material (MB) options - there are 6 at maximum /// -------------------- MB (outer [1]) --------------- /// | MB [2] NEC MB [3] | B | MB [4] PEC MB [5] | /// -------------------- MB (inner [0]) --------------- @@ -510,7 +510,7 @@ class CylinderVolumeBuilder : public ITrackingVolumeBuilder { /// Constructor /// - /// @param [in] cvbConfig is the configuraiton struct to steer the builder + /// @param [in] cvbConfig is the configuration struct to steer the builder /// @param [in] logger logging instance CylinderVolumeBuilder(const Config& cvbConfig, std::unique_ptr logger = getDefaultLogger( @@ -544,7 +544,7 @@ class CylinderVolumeBuilder : public ITrackingVolumeBuilder { /// set logging instance /// - /// @param [in] newLogger is the logging istance to be set + /// @param [in] newLogger is the logging instance to be set void setLogger(std::unique_ptr newLogger); /// Analyze the config to gather needed dimension diff --git a/Core/include/Acts/Geometry/CylinderVolumeHelper.hpp b/Core/include/Acts/Geometry/CylinderVolumeHelper.hpp index 0bf01dd1523..5762552d7ec 100644 --- a/Core/include/Acts/Geometry/CylinderVolumeHelper.hpp +++ b/Core/include/Acts/Geometry/CylinderVolumeHelper.hpp @@ -70,7 +70,7 @@ class CylinderVolumeHelper : public ITrackingVolumeHelper { /// together with the volume enevlope parameters /// @param volumeMaterial material properties for this TrackingVolume /// @param volumeBounds: confinement of this TrackingVolume - /// @param mtvVector (optiona) Vector of confined TrackingVolumes + /// @param mtvVector (optional) Vector of confined TrackingVolumes /// @param transform (optional) placement of this TrackingVolume /// @param volumeName volume name to be given /// @param bType (optional) BinningType - arbitrary(default) or equidistant @@ -118,7 +118,7 @@ class CylinderVolumeHelper : public ITrackingVolumeHelper { /// @param rMax maximum radius /// @param zMin minimum z /// @param zMax maximum z - /// @param materialLayers number of material layers (aequidistant binning) + /// @param materialLayers number of material layers (equidistant binning) /// @param cylinder type of layers /// @param volumeName volume name to be given /// @@ -167,7 +167,7 @@ class CylinderVolumeHelper : public ITrackingVolumeHelper { /// Set configuration method /// - /// @param cvhConfig is the configurtion struct assigned + /// @param cvhConfig is the configuration struct assigned void setConfiguration(const Config& cvhConfig); /// Get configuration method @@ -175,7 +175,7 @@ class CylinderVolumeHelper : public ITrackingVolumeHelper { /// Set logging instance /// - /// @param newLogger is the logger isntance to be set + /// @param newLogger is the logger instance to be set void setLogger(std::unique_ptr newLogger); protected: @@ -217,7 +217,7 @@ class CylinderVolumeHelper : public ITrackingVolumeHelper { /// @param rBinned a boolean indicating if it is binned in r /// @param rMin the minimum radius of the volume /// @param rGlueMin the minimum glue radius (@todo check and document) - /// @param rMax the maximim radius of the volume + /// @param rMax the maximum radius of the volume /// @param zMin the minimum z extend of the volume /// @param zMax the maximum z extend of the volume bool interGlueTrackingVolume(const GeometryContext& gctx, @@ -234,7 +234,7 @@ class CylinderVolumeHelper : public ITrackingVolumeHelper { /// @param faceTwo is the second boundary face of the glue process /// @param rMin the minimum radius of the volume /// @param rGlueMin the minimum glue radius (@todo check and document) - /// @param rMax the maximim radius of the volume + /// @param rMax the maximum radius of the volume /// @param zMin the minimum z extend of the volume /// @param zMax the maximum z extend of the volume void glueTrackingVolumes(const GeometryContext& gctx, diff --git a/Core/include/Acts/Geometry/Extent.hpp b/Core/include/Acts/Geometry/Extent.hpp index f4aed1efc15..63e6d3ddb73 100644 --- a/Core/include/Acts/Geometry/Extent.hpp +++ b/Core/include/Acts/Geometry/Extent.hpp @@ -32,7 +32,7 @@ constexpr ExtentEnvelope zeroEnvelopes = { zeroEnvelope, zeroEnvelope, zeroEnvelope, zeroEnvelope, zeroEnvelope, zeroEnvelope, zeroEnvelope, zeroEnvelope}; -/// A class representing the geometric extent of an object in its possbile +/// A class representing the geometric extent of an object in its possible /// dimensions, these can be all dimensions that are described as BinningValues /// /// The extent object can have an optional envelope in all of those values @@ -154,7 +154,7 @@ class Extent { /// Return the histogram store /// - /// The histogram stroe can be used for automated binning detection + /// The histogram store can be used for automated binning detection const std::array, binValues>& valueHistograms() const; /// Access the minimum parameter diff --git a/Core/include/Acts/Geometry/GenericApproachDescriptor.hpp b/Core/include/Acts/Geometry/GenericApproachDescriptor.hpp index 570b9b027ac..fae586b6b01 100644 --- a/Core/include/Acts/Geometry/GenericApproachDescriptor.hpp +++ b/Core/include/Acts/Geometry/GenericApproachDescriptor.hpp @@ -51,10 +51,10 @@ class GenericApproachDescriptor : public ApproachDescriptor { /// @brief Register the Layer to the surfaces /// - /// @param lay is the layer to be registerd + /// @param lay is the layer to be registered void registerLayer(const Layer& lay) override; - /// Get the aproach surface to the layer + /// Get the approach surface to the layer /// /// @param gctx The current geometry context object, e.g. alignment /// @param position The global position to start the approach from diff --git a/Core/include/Acts/Geometry/GeometryHierarchyMap.hpp b/Core/include/Acts/Geometry/GeometryHierarchyMap.hpp index edb01254f92..da8b8f4928e 100644 --- a/Core/include/Acts/Geometry/GeometryHierarchyMap.hpp +++ b/Core/include/Acts/Geometry/GeometryHierarchyMap.hpp @@ -106,7 +106,7 @@ class GeometryHierarchyMap { /// /// This can be either from the element matching exactly to the given geometry /// id, if it exists, or from the element for the next available higher level - /// within the geometry hierachy. + /// within the geometry hierarchy. /// /// @param id geometry identifier for which information is requested /// @retval iterator to an existing value diff --git a/Core/include/Acts/Geometry/GeometryObjectSorter.hpp b/Core/include/Acts/Geometry/GeometryObjectSorter.hpp index 37cc4ce6ae0..a2f6ec0c095 100644 --- a/Core/include/Acts/Geometry/GeometryObjectSorter.hpp +++ b/Core/include/Acts/Geometry/GeometryObjectSorter.hpp @@ -33,7 +33,7 @@ class ObjectSorterT { /// @param one first object /// @param two second object /// - /// @return boolen indicator + /// @return boolean indicator bool operator()(T one, T two) const { using Acts::VectorHelpers::eta; using Acts::VectorHelpers::perp; @@ -98,7 +98,7 @@ class DistanceSorterT { /// @tparam one first object /// @tparam two second object /// - /// @return boolen indicator + /// @return boolean indicator bool operator()(T one, T two) const { using Acts::VectorHelpers::eta; using Acts::VectorHelpers::perp; @@ -178,7 +178,7 @@ class GeometryObjectSorterT { /// @tparam one first object /// @tparam two second object /// - /// @return boolen indicator + /// @return boolean indicator bool operator()(T one, T two) const { // get the pos one / pos two Vector3 posOne = diff --git a/Core/include/Acts/Geometry/GlueVolumesDescriptor.hpp b/Core/include/Acts/Geometry/GlueVolumesDescriptor.hpp index 6379bb9ac09..d04fdfe5050 100644 --- a/Core/include/Acts/Geometry/GlueVolumesDescriptor.hpp +++ b/Core/include/Acts/Geometry/GlueVolumesDescriptor.hpp @@ -42,7 +42,7 @@ class GlueVolumesDescriptor { const std::map>& gvs); - /// Desctructor + /// Destructor ~GlueVolumesDescriptor() = default; /// Register the volumes /// diff --git a/Core/include/Acts/Geometry/IConfinedTrackingVolumeBuilder.hpp b/Core/include/Acts/Geometry/IConfinedTrackingVolumeBuilder.hpp index 46bd3cc1d72..22d44a5f4e1 100644 --- a/Core/include/Acts/Geometry/IConfinedTrackingVolumeBuilder.hpp +++ b/Core/include/Acts/Geometry/IConfinedTrackingVolumeBuilder.hpp @@ -26,7 +26,7 @@ class IConfinedTrackingVolumeBuilder { /// Interface for constructing a vector of confined TrackingVolumes virtual MutableTrackingVolumeVector centralVolumes() const = 0; - /// Interface for retreiving the identification string of the confined volumes + /// Interface for retrieving the identification string of the confined volumes virtual const std::string& identification() const = 0; }; diff --git a/Core/include/Acts/Geometry/ILayerArrayCreator.hpp b/Core/include/Acts/Geometry/ILayerArrayCreator.hpp index f95be794c2b..20fe6e5210f 100644 --- a/Core/include/Acts/Geometry/ILayerArrayCreator.hpp +++ b/Core/include/Acts/Geometry/ILayerArrayCreator.hpp @@ -46,7 +46,7 @@ class ILayerArrayCreator { /// @param btype is the binning type /// @param bvalue is the value in which the binning should be done /// - /// @return unqiue pointer to a new LayerArray + /// @return unique pointer to a new LayerArray virtual std::unique_ptr layerArray( const GeometryContext& gctx, const LayerVector& layers, double min, double max, BinningType btype = arbitrary, diff --git a/Core/include/Acts/Geometry/ILayerBuilder.hpp b/Core/include/Acts/Geometry/ILayerBuilder.hpp index 19bc78372a8..41ed9d2ee2e 100644 --- a/Core/include/Acts/Geometry/ILayerBuilder.hpp +++ b/Core/include/Acts/Geometry/ILayerBuilder.hpp @@ -33,7 +33,7 @@ class ILayerBuilder { /// LayerBuilder interface method /// - /// @param gctx ist the geometry context under + /// @param gctx is the geometry context under /// which the geometry is built /// /// @return the layers at negative side @@ -42,7 +42,7 @@ class ILayerBuilder { /// LayerBuilder interface method /// - /// @param gctx ist the geometry context under + /// @param gctx is the geometry context under /// which the geometry is built /// /// @return the layers at the central sector @@ -51,7 +51,7 @@ class ILayerBuilder { /// LayerBuilder interface method /// - /// @param gctx ist the geometry context under + /// @param gctx is the geometry context under /// which the geometry is built /// /// @return the layers at positive side diff --git a/Core/include/Acts/Geometry/ITrackingGeometryBuilder.hpp b/Core/include/Acts/Geometry/ITrackingGeometryBuilder.hpp index 1744e881830..ddfd2a3e169 100644 --- a/Core/include/Acts/Geometry/ITrackingGeometryBuilder.hpp +++ b/Core/include/Acts/Geometry/ITrackingGeometryBuilder.hpp @@ -17,7 +17,7 @@ class TrackingGeometry; /// @class ITrackingGeometryBuilder /// /// Interface class for the TrackingGeometry building, -/// this is used by the TrackingGeometrySvc to build the geoemtry. +/// this is used by the TrackingGeometrySvc to build the geometry. /// /// The TrackingGeometry is written to the detector store and thus not created /// as a std::shared_ptr. @@ -30,9 +30,9 @@ class ITrackingGeometryBuilder { /// Virtual destructor virtual ~ITrackingGeometryBuilder() = default; - /// TrackingGeometry Interface methode + /// TrackingGeometry Interface method /// - /// @param gctx ist the geometry context for witch the geometry is built + /// @param gctx is the geometry context for witch the geometry is built /// /// @return unique pointer to a newly created TrackingGeometry virtual std::unique_ptr trackingGeometry( diff --git a/Core/include/Acts/Geometry/ITrackingVolumeHelper.hpp b/Core/include/Acts/Geometry/ITrackingVolumeHelper.hpp index f41d7b254dc..b7d43499877 100644 --- a/Core/include/Acts/Geometry/ITrackingVolumeHelper.hpp +++ b/Core/include/Acts/Geometry/ITrackingVolumeHelper.hpp @@ -97,7 +97,7 @@ class ITrackingVolumeHelper { /// @param volumeMaterial material properties for this TrackingVolume /// @param loc0Min, loc0Max, loc1Min, loc1Max : local position in space, /// this TrackingVolume is restricted to Translation only - /// @param materialLayers number of material layers (aequidistant binning) + /// @param materialLayers number of material layers (equidistant binning) /// @param cylinder type of layers /// @param volumeName volume name to be given /// diff --git a/Core/include/Acts/Geometry/KDTreeTrackingGeometryBuilder.hpp b/Core/include/Acts/Geometry/KDTreeTrackingGeometryBuilder.hpp index 231c32019ee..2e650841c53 100644 --- a/Core/include/Acts/Geometry/KDTreeTrackingGeometryBuilder.hpp +++ b/Core/include/Acts/Geometry/KDTreeTrackingGeometryBuilder.hpp @@ -110,7 +110,7 @@ class KDTreeTrackingGeometryBuilder : public ITrackingGeometryBuilder { /// @param cCache is a cache used to extract the built detector elements /// @param gctx is the current geometry context at building /// @param kdt is the pre-filled kdt tree for the surface query - /// @param plVolume the proto volume representaion a layer to be translated + /// @param plVolume the proto volume representation a layer to be translated /// @param indent is a screen output indentation /// /// @return a new tracking volume diff --git a/Core/include/Acts/Geometry/Layer.hpp b/Core/include/Acts/Geometry/Layer.hpp index 5f19b9f2298..1bf57474581 100644 --- a/Core/include/Acts/Geometry/Layer.hpp +++ b/Core/include/Acts/Geometry/Layer.hpp @@ -66,7 +66,7 @@ enum LayerType { navigation = -1, passive = 0, active = 1 }; /// subSurfaces. /// A pointer to the TrackingVolume (can only be set by such) /// An active/passive code : -/// 0 - activ +/// 0 - active /// 1 - passive /// [....] - other /// @@ -126,7 +126,7 @@ class Layer : public virtual GeometryObject { /// @note using isOnSurface() with Layer specific tolerance /// /// @param gctx The current geometry context object, e.g. alignment - /// @param position is the gobal position to be checked + /// @param position is the global position to be checked /// @param bcheck is the boundary check directive /// /// @return boolean that indicates success of the operation @@ -275,7 +275,7 @@ class Layer : public virtual GeometryObject { /// Private helper method to close the geometry /// - it will assign material to the surfaces if needed /// - it will set the layer geometry ID for a unique identification - /// - it will also register the internal sub strucutre + /// - it will also register the internal sub structure /// /// @param materialDecorator is a decorator that assigns /// optionally the surface material to where they belong diff --git a/Core/include/Acts/Geometry/LayerArrayCreator.hpp b/Core/include/Acts/Geometry/LayerArrayCreator.hpp index 21adc4dbb49..6f87d56b1bb 100644 --- a/Core/include/Acts/Geometry/LayerArrayCreator.hpp +++ b/Core/include/Acts/Geometry/LayerArrayCreator.hpp @@ -49,7 +49,7 @@ class LayerArrayCreator : public ILayerArrayCreator { /// LayerArrayCreator interface method /// - /// @param gctx ist the geometry context for witch the array is built + /// @param gctx is the geometry context for witch the array is built /// @param layersInput are the layers to be moved into an array /// @param min is the minimum value for binning /// @param max is the maximum value for binning diff --git a/Core/include/Acts/Geometry/LayerCreator.hpp b/Core/include/Acts/Geometry/LayerCreator.hpp index f3c57b038e6..607a8010624 100644 --- a/Core/include/Acts/Geometry/LayerCreator.hpp +++ b/Core/include/Acts/Geometry/LayerCreator.hpp @@ -67,7 +67,7 @@ class LayerCreator { /// returning a cylindrical layer /// - /// @param gctx ist the geometry context with which the geometry is built + /// @param gctx is the geometry context with which the geometry is built /// @param surfaces is the vector of pointers to sensitive surfaces /// represented by this layer /// @pre the pointers to the sensitive surfaces in the surfaces vectors all @@ -91,7 +91,7 @@ class LayerCreator { /// returning a cylindrical layer /// - /// @param gctx ist the geometry context with which the geometry is built + /// @param gctx is the geometry context with which the geometry is built /// @param surfaces is the vector of pointers to sensitive surfaces /// represented by this layer /// @pre the pointers to the sensitive surfaces in the surfaces vectors all @@ -116,7 +116,7 @@ class LayerCreator { /// returning a disc layer /// - /// @param gctx ist the geometry context with which the geometry is built + /// @param gctx is the geometry context with which the geometry is built /// @param surfaces is the vector of pointers to sensitive surfaces /// represented by this layer /// @pre the pointers to the sensitive surfaces in the surfaces vectors all @@ -140,7 +140,7 @@ class LayerCreator { /// returning a disc layer /// - /// @param gctx ist the geometry context with which the geometry is built + /// @param gctx is the geometry context with which the geometry is built /// @param surfaces is the vector of pointers to sensitive surfaces /// represented by this layer /// @pre the pointers to the sensitive surfaces in the surfaces vectors all @@ -165,7 +165,7 @@ class LayerCreator { /// returning a plane layer /// - /// @param gctx ist the geometry context with which the geometry is built + /// @param gctx is the geometry context with which the geometry is built /// @param [in] surfaces is the vector of pointers to sensitive surfaces /// represented by this layer /// @pre the pointers to the sensitive surfaces in the surfaces vectors all @@ -218,7 +218,7 @@ class LayerCreator { /// configuration object Config m_cfg; - /// Private acces method to the logger + /// Private access method to the logger const Logger& logger() const { return *m_logger; } /// logging instance diff --git a/Core/include/Acts/Geometry/NavigationLayer.hpp b/Core/include/Acts/Geometry/NavigationLayer.hpp index d9ebda76c54..1c69b0637aa 100644 --- a/Core/include/Acts/Geometry/NavigationLayer.hpp +++ b/Core/include/Acts/Geometry/NavigationLayer.hpp @@ -79,7 +79,7 @@ class NavigationLayer : public Layer { bool isOnLayer(const GeometryContext& gctx, const Vector3& gp, const BoundaryCheck& bcheck = true) const final; - /// Accept layer according to the following colelction directives + /// Accept layer according to the following collection directives /// /// @param resolveSensitive is the prescription to find the sensitive surfaces /// @param resolveMaterial is the precription to find material surfaces diff --git a/Core/include/Acts/Geometry/PassiveLayerBuilder.hpp b/Core/include/Acts/Geometry/PassiveLayerBuilder.hpp index 4a19d56db4d..e22f6a94115 100644 --- a/Core/include/Acts/Geometry/PassiveLayerBuilder.hpp +++ b/Core/include/Acts/Geometry/PassiveLayerBuilder.hpp @@ -63,7 +63,7 @@ class PassiveLayerBuilder : public ILayerBuilder { /// LayerBuilder interface method /// - /// @param gctx ist the geometry context under + /// @param gctx is the geometry context under /// which the geometry is built /// /// @return the layers at negative side @@ -71,7 +71,7 @@ class PassiveLayerBuilder : public ILayerBuilder { /// LayerBuilder interface method /// - /// @param gctx ist the geometry context under + /// @param gctx is the geometry context under /// which the geometry is built /// /// @return the layers at the central sector @@ -79,7 +79,7 @@ class PassiveLayerBuilder : public ILayerBuilder { /// LayerBuilder interface method /// - /// @param gctx ist the geometry context under + /// @param gctx is the geometry context under /// which the geometry is built /// /// @return the layers at positive side @@ -111,7 +111,7 @@ class PassiveLayerBuilder : public ILayerBuilder { private: /// Helper interface method /// - /// @param gctx ist the geometry context under + /// @param gctx is the geometry context under /// which the geometry is built /// @param side is the side of the layer to be built /// diff --git a/Core/include/Acts/Geometry/SurfaceArrayCreator.hpp b/Core/include/Acts/Geometry/SurfaceArrayCreator.hpp index f5ae5bcf3ea..403c2ae87ba 100644 --- a/Core/include/Acts/Geometry/SurfaceArrayCreator.hpp +++ b/Core/include/Acts/Geometry/SurfaceArrayCreator.hpp @@ -117,7 +117,7 @@ class SurfaceArrayCreator { /// to be ordered on the cylinder /// @pre the pointers to the sensitive surfaces in the surfaces vectors all /// need to be valid, since no check is performed - /// @param [in] gctx The gometry context fro this building call + /// @param [in] gctx The gometry context for this building call /// @param protoLayerOpt The proto layer containing the layer size /// @param binsPhi is the number of bins in phi for the surfaces /// @param binsZ is the number of bin in Z for the surfaces @@ -140,10 +140,10 @@ class SurfaceArrayCreator { /// to be ordered on the cylinder /// @pre the pointers to the sensitive surfaces in the surfaces vectors all /// need to be valid, since no check is performed - /// @param [in] gctx The gometry context fro this building call + /// @param [in] gctx The gometry context for this building call /// @param protoLayerOpt The proto layer containing the layer size - /// @param bTypePhi the binning type in phi direction (equidistant/aribtrary) - /// @param bTypeZ the binning type in z direction (equidistant/aribtrary) + /// @param bTypePhi the binning type in phi direction (equidistant/arbitrary) + /// @param bTypeZ the binning type in z direction (equidistant/arbitrary) /// @param transform is the (optional) additional transform applied /// /// @return a unique pointer a new SurfaceArray @@ -163,7 +163,7 @@ class SurfaceArrayCreator { /// @pre the pointers to the sensitive surfaces in the surfaces vectors all /// need to be valid, since no check is performed /// @warning This function requires the disc aligned with the z-axis - /// @param [in] gctx The gometry context fro this building call + /// @param [in] gctx The gometry context for this building call /// @param protoLayerOpt The proto layer containing the layer size /// @param binsPhi is the number of bins in phi for the surfaces /// @param binsR is the number of bin in R for the surfaces @@ -186,10 +186,10 @@ class SurfaceArrayCreator { /// @pre the pointers to the sensitive surfaces in the surfaces vectors all /// need to be valid, since no check is performed /// @warning This function requires the disc aligned with the z-axis - /// @param [in] gctx The gometry context fro this building call + /// @param [in] gctx The gometry context for this building call /// @param protoLayerOpt The proto layer containing the layer size - /// @param bTypeR the binning type in r direction (equidistant/aribtrary) - /// @param bTypePhi the binning type in phi direction (equidistant/aribtrary) + /// @param bTypeR the binning type in r direction (equidistant/arbitrary) + /// @param bTypePhi the binning type in phi direction (equidistant/arbitrary) /// @param transform is the (optional) additional transform applied /// /// @return a unique pointer a new SurfaceArray @@ -206,7 +206,7 @@ class SurfaceArrayCreator { /// SurfaceArrayCreator interface method /// - create an array on a plane /// - /// @param [in] gctx The gometry context fro this building call + /// @param [in] gctx The gometry context for this building call /// @param [in] surfaces is the vector of pointers to sensitive surfaces /// to be ordered on the plane /// @pre the pointers to the sensitive surfaces in the surfaces vectors all @@ -229,7 +229,7 @@ class SurfaceArrayCreator { std::optional protoLayerOpt = std::nullopt, const Transform3& transform = Transform3::Identity()) const; - /// Static check funtion for surface equivalent + /// Static check function for surface equivalent /// /// @param [in] gctx the geometry context for this check /// @param bValue the binning value for the binning diff --git a/Core/include/Acts/Geometry/TrackingGeometry.hpp b/Core/include/Acts/Geometry/TrackingGeometry.hpp index 12ad6c6b7a7..e84590e6f30 100644 --- a/Core/include/Acts/Geometry/TrackingGeometry.hpp +++ b/Core/include/Acts/Geometry/TrackingGeometry.hpp @@ -38,7 +38,7 @@ using MutableTrackingVolumePtr = std::shared_ptr; /// /// It enables both, a global search for an asociatedVolume /// (respectively, if existing, a global search of an associated Layer or the -/// next associated Layer), such as a continous navigation by BoundarySurfaces +/// next associated Layer), such as a continuous navigation by BoundarySurfaces /// between the confined TrackingVolumes. class TrackingGeometry { /// Give the GeometryBuilder friend rights diff --git a/Core/include/Acts/Geometry/TrackingVolume.hpp b/Core/include/Acts/Geometry/TrackingVolume.hpp index daceb5e12e0..7629e0745bb 100644 --- a/Core/include/Acts/Geometry/TrackingVolume.hpp +++ b/Core/include/Acts/Geometry/TrackingVolume.hpp @@ -259,7 +259,7 @@ class TrackingVolume : public Volume { /// @param visitor The callable. Will be called for each sensitive surface /// that is found /// - /// If a context is needed for the vist, the vistitor has to provide this + /// If a context is needed for the visit, the vistitor has to provide this /// e.g. as a private member template void visitSurfaces(visitor_t&& visitor) const { @@ -324,7 +324,7 @@ class TrackingVolume : public Volume { /// @param gctx The current geometry context object, e.g. alignment /// @param bsfMine is the boundary face indicater where to glue /// @param neighbor is the TrackingVolume to be glued - /// @param bsfNeighbor is the boudnary surface of the neighbor + /// @param bsfNeighbor is the boundary surface of the neighbor void glueTrackingVolume(const GeometryContext& gctx, BoundarySurfaceFace bsfMine, TrackingVolume* neighbor, BoundarySurfaceFace bsfNeighbor); @@ -336,7 +336,7 @@ class TrackingVolume : public Volume { /// @param gctx The current geometry context object, e.g. alignment /// @param bsfMine is the boundary face indicater where to glue /// @param neighbors are the TrackingVolumes to be glued - /// @param bsfNeighbor are the boudnary surface of the neighbors + /// @param bsfNeighbor are the boundary surface of the neighbors void glueTrackingVolumes( const GeometryContext& gctx, BoundarySurfaceFace bsfMine, const std::shared_ptr& neighbors, @@ -345,7 +345,7 @@ class TrackingVolume : public Volume { /// Provide a new BoundarySurface from the glueing /// /// @param bsf is the boundary face indicater where to glue - /// @param bs is the new boudnary surface + /// @param bs is the new boundary surface /// @param checkmaterial is a flag how to deal with material, if true: /// - if the old boundary surface had a material description /// but the new one has not, keep the current one diff --git a/Core/include/Acts/Geometry/TrapezoidVolumeBounds.hpp b/Core/include/Acts/Geometry/TrapezoidVolumeBounds.hpp index e976efd6e30..02d638af0f5 100644 --- a/Core/include/Acts/Geometry/TrapezoidVolumeBounds.hpp +++ b/Core/include/Acts/Geometry/TrapezoidVolumeBounds.hpp @@ -49,7 +49,7 @@ class TrapezoidBounds; /// class TrapezoidVolumeBounds : public VolumeBounds { public: - /// @enum BoundValues for acces / streaming + /// @enum BoundValues for access / streaming enum BoundValues : unsigned int { eHalfLengthXnegY = 0, //!< halflength in x at negative y eHalfLengthXposY = 1, //!< halflength in x at positive y @@ -76,8 +76,8 @@ class TrapezoidVolumeBounds : public VolumeBounds { /// @param minhalex is the half length in x at minimal y /// @param haley is the half length in y /// @param halez is the half length in z - /// @param alpha is the openeing angle at -x,-y - /// @param beta is the openeing angle at +x,-y + /// @param alpha is the opening angle at -x,-y + /// @param beta is the opening angle at +x,-y TrapezoidVolumeBounds(double minhalex, double haley, double halez, double alpha, double beta) noexcept(false); @@ -163,7 +163,7 @@ class TrapezoidVolumeBounds : public VolumeBounds { /// Helper method to create the surface bounds void buildSurfaceBounds(); - /// Templated dump methos + /// Templated dump methods /// @tparam stream_t The type of the stream for dumping /// @param dt The stream object template diff --git a/Core/include/Acts/MagneticField/SolenoidBField.hpp b/Core/include/Acts/MagneticField/SolenoidBField.hpp index 01b040f22bf..a3ac9499226 100644 --- a/Core/include/Acts/MagneticField/SolenoidBField.hpp +++ b/Core/include/Acts/MagneticField/SolenoidBField.hpp @@ -90,7 +90,7 @@ class SolenoidBField final : public MagneticFieldProvider { double bMagCenter; }; - /// @brief the constructur with a shared pointer + /// @brief the constructor with a shared pointer /// @note since it is a shared field, we enforce it to be const /// @tparam bField is the shared BField to be stored SolenoidBField(Config config); diff --git a/Core/include/Acts/Material/AccumulatedMaterialSlab.hpp b/Core/include/Acts/Material/AccumulatedMaterialSlab.hpp index 5f8b16720dd..4f41a2e50a7 100644 --- a/Core/include/Acts/Material/AccumulatedMaterialSlab.hpp +++ b/Core/include/Acts/Material/AccumulatedMaterialSlab.hpp @@ -68,7 +68,7 @@ class AccumulatedMaterialSlab { /// Each track contributes equally to the total average regardless of its /// measured path within the material. An empty per-track store, i.e. /// vanishing per-track material thickness, does not contribute to the total - /// unless explicitely requested. + /// unless explicitly requested. void trackAverage(bool useEmptyTrack = false); /// Return the average material properties from all accumulated tracks. diff --git a/Core/include/Acts/Material/AccumulatedSurfaceMaterial.hpp b/Core/include/Acts/Material/AccumulatedSurfaceMaterial.hpp index d5ce9a7c4dd..173a84f7fa3 100644 --- a/Core/include/Acts/Material/AccumulatedSurfaceMaterial.hpp +++ b/Core/include/Acts/Material/AccumulatedSurfaceMaterial.hpp @@ -80,7 +80,7 @@ class AccumulatedSurfaceMaterial { /// Return the BinUtility const BinUtility& binUtility() const; - /// Assign a material properites object + /// Assign a material properties object /// /// @param lp local position for the bin assignment /// @param mp material properties to be assigned @@ -90,7 +90,7 @@ class AccumulatedSurfaceMaterial { std::array accumulate(const Vector2& lp, const MaterialSlab& mp, double pathCorrection = 1.); - /// Assign a material properites object + /// Assign a material properties object /// /// @param gp global position for the bin assignment /// @param mp material properties to be assigned diff --git a/Core/include/Acts/Material/HomogeneousSurfaceMaterial.hpp b/Core/include/Acts/Material/HomogeneousSurfaceMaterial.hpp index 780ac8bc2f3..584c745685b 100644 --- a/Core/include/Acts/Material/HomogeneousSurfaceMaterial.hpp +++ b/Core/include/Acts/Material/HomogeneousSurfaceMaterial.hpp @@ -19,7 +19,7 @@ namespace Acts { /// @class HomogeneousSurfaceMaterial /// -/// It extends the ISurfaceMaterial virutal base class to describe +/// It extends the ISurfaceMaterial virtual base class to describe /// a simple homogeneous material on a surface class HomogeneousSurfaceMaterial : public ISurfaceMaterial { public: @@ -88,7 +88,7 @@ class HomogeneousSurfaceMaterial : public ISurfaceMaterial { /// The inherited methods - for MaterialSlab access using ISurfaceMaterial::materialSlab; - /// The interited methods - for scale access + /// The inherited methods - for scale access using ISurfaceMaterial::factor; /// Output Method for std::ostream diff --git a/Core/include/Acts/Material/IMaterialDecorator.hpp b/Core/include/Acts/Material/IMaterialDecorator.hpp index 3abed78b023..3b6d390210b 100644 --- a/Core/include/Acts/Material/IMaterialDecorator.hpp +++ b/Core/include/Acts/Material/IMaterialDecorator.hpp @@ -18,7 +18,7 @@ class TrackingVolume; /// Virtual base class for decorators that allow to load /// material onto a TrackingGeometry. The geometry allows material /// to be assigned either to surfaces or to volumes, hence there are -/// two decorate interface methots. +/// two decorate interface methods. /// class IMaterialDecorator { public: diff --git a/Core/include/Acts/Material/ISurfaceMaterial.hpp b/Core/include/Acts/Material/ISurfaceMaterial.hpp index 7109813ba6f..4aca9cd42dd 100644 --- a/Core/include/Acts/Material/ISurfaceMaterial.hpp +++ b/Core/include/Acts/Material/ISurfaceMaterial.hpp @@ -27,7 +27,7 @@ enum MappingType { PreMapping = -1, Default = 0, PostMapping = 1, Sensor = 2 }; /// Virtual base class of surface based material description /// /// MaterialSlab that are associated to a surface, -/// extended by certain special representations (binned, homogenous) +/// extended by certain special representations (binned, homogeneous) /// class ISurfaceMaterial { public: diff --git a/Core/include/Acts/Material/IVolumeMaterial.hpp b/Core/include/Acts/Material/IVolumeMaterial.hpp index a35ad0f2ffc..5f1d18f50e6 100644 --- a/Core/include/Acts/Material/IVolumeMaterial.hpp +++ b/Core/include/Acts/Material/IVolumeMaterial.hpp @@ -19,7 +19,7 @@ class Material; /// /// Virtual base class of volume material description // -/// Material associated with a Volume (homogenous, binned, interpolated) +/// Material associated with a Volume (homogeneous, binned, interpolated) class IVolumeMaterial { public: /// Virtual Destructor diff --git a/Core/include/Acts/Material/Interactions.hpp b/Core/include/Acts/Material/Interactions.hpp index bd028921ee0..73854a4efd9 100644 --- a/Core/include/Acts/Material/Interactions.hpp +++ b/Core/include/Acts/Material/Interactions.hpp @@ -81,7 +81,7 @@ float deriveEnergyLossLandauQOverP(const MaterialSlab& slab, int pdg, float m, /// /// @see computeEnergyLossBethe for parameters description /// -/// This is the sigma paramter of a Gaussian distribution with the same +/// This is the sigma parameter of a Gaussian distribution with the same /// full-width-half-maximum as the Landau-Vavilov-Bichsel distribution. The /// computations are valid for intermediate particle energies. float computeEnergyLossLandauSigma(const MaterialSlab& slab, int pdg, float m, diff --git a/Core/include/Acts/Material/MaterialGridHelper.hpp b/Core/include/Acts/Material/MaterialGridHelper.hpp index 4e800a11426..69fe62260aa 100644 --- a/Core/include/Acts/Material/MaterialGridHelper.hpp +++ b/Core/include/Acts/Material/MaterialGridHelper.hpp @@ -76,7 +76,7 @@ std::function globalToLocalFromBin( Acts::BinningValue& type); /// @brief Create a 2DGrid using a BinUtility. -/// Also determine the coresponding global to local transform and grid mapping +/// Also determine the corresponding global to local transform and grid mapping /// function /// /// @param [in] bins BinUtility of the volume to be mapped @@ -88,7 +88,7 @@ Grid2D createGrid2D( std::function& transfoGlobalToLocal); /// @brief Create a 3DGrid using a BinUtility. -/// Also determine the coresponding global to local transform and grid mapping +/// Also determine the corresponding global to local transform and grid mapping /// function /// /// @param [in] bins BinUtility of the volume to be mapped diff --git a/Core/include/Acts/Material/MaterialInteraction.hpp b/Core/include/Acts/Material/MaterialInteraction.hpp index b421be00b39..bc59a08de2c 100644 --- a/Core/include/Acts/Material/MaterialInteraction.hpp +++ b/Core/include/Acts/Material/MaterialInteraction.hpp @@ -19,7 +19,7 @@ class TrackingVolume; /// @brief The Material interaction struct /// It records the surface and the passed material -/// This is only nessecary recorded when configured +/// This is only necessary recorded when configured struct MaterialInteraction { /// The particle position at the interaction. Vector3 position = Vector3(0., 0., 0); @@ -35,15 +35,15 @@ struct MaterialInteraction { double sigmaTheta2 = 0.0; /// Expected q/p variance due to the interactions. double sigmaQoP2 = 0.0; - /// The position where the interaction occured. + /// The position where the interaction occurred. Vector3 intersection = Vector3(0., 0., 0); - /// The ID where the interaction occured. + /// The ID where the interaction occurred. GeometryIdentifier intersectionID; - /// The surface where the interaction occured. + /// The surface where the interaction occurred. const Surface* surface = nullptr; - /// The volume where the interaction occured. + /// The volume where the interaction occurred. const TrackingVolume* volume = nullptr; - /// Update the volume step to implment the proper step size + /// Update the volume step to implement the proper step size bool updatedVolumeStep = false; /// The path correction factor due to non-zero incidence on the surface. double pathCorrection = 1.; diff --git a/Core/include/Acts/Material/ProtoSurfaceMaterial.hpp b/Core/include/Acts/Material/ProtoSurfaceMaterial.hpp index 60fd53342f9..1f5aff89e8a 100644 --- a/Core/include/Acts/Material/ProtoSurfaceMaterial.hpp +++ b/Core/include/Acts/Material/ProtoSurfaceMaterial.hpp @@ -29,7 +29,7 @@ namespace Acts { class ProtoSurfaceMaterial : public ISurfaceMaterial { public: - /// Constructor without BinUtility - homogenous material + /// Constructor without BinUtility - homogeneous material ProtoSurfaceMaterial() = default; /// Constructor with BinUtility - multidimensional material @@ -40,12 +40,12 @@ class ProtoSurfaceMaterial : public ISurfaceMaterial { ProtoSurfaceMaterial(const BinUtility& binUtility, MappingType mappingType = MappingType::Default); - /// Copy constuctor + /// Copy constructor /// /// @param smproxy The source proxy ProtoSurfaceMaterial(const ProtoSurfaceMaterial& smproxy) = default; - /// Copy move constuctor + /// Copy move constructor /// /// @param smproxy The source proxy ProtoSurfaceMaterial(ProtoSurfaceMaterial&& smproxy) = default; @@ -59,7 +59,7 @@ class ProtoSurfaceMaterial : public ISurfaceMaterial { ProtoSurfaceMaterial& operator=(const ProtoSurfaceMaterial& smproxy) = default; - /// Assigment move operator + /// Assignment move operator /// /// @param smproxy The source proxy ProtoSurfaceMaterial& operator=(ProtoSurfaceMaterial&& smproxy) = default; @@ -91,7 +91,7 @@ class ProtoSurfaceMaterial : public ISurfaceMaterial { /// Direct access via bins to the MaterialSlab /// /// @param ib0 indicates the first bin - /// @param ib1 indicates the seconf bin + /// @param ib1 indicates the second bin /// /// @return will return dummy material const MaterialSlab& materialSlab(size_t ib0, size_t ib1) const final; diff --git a/Core/include/Acts/Material/ProtoVolumeMaterial.hpp b/Core/include/Acts/Material/ProtoVolumeMaterial.hpp index 7760cf3e387..fb0a02dfa8f 100644 --- a/Core/include/Acts/Material/ProtoVolumeMaterial.hpp +++ b/Core/include/Acts/Material/ProtoVolumeMaterial.hpp @@ -28,7 +28,7 @@ namespace Acts { class ProtoVolumeMaterial : public IVolumeMaterial { public: - /// Constructor without BinUtility - homogenous material + /// Constructor without BinUtility - homogeneous material ProtoVolumeMaterial() = default; /// Constructor with BinUtility - multidimensional material @@ -37,12 +37,12 @@ class ProtoVolumeMaterial : public IVolumeMaterial { /// and binning of the material on the volume ProtoVolumeMaterial(const BinUtility& binUtility); - /// Copy constuctor + /// Copy constructor /// /// @param vmproxy The source proxy ProtoVolumeMaterial(const ProtoVolumeMaterial& vmproxy) = default; - /// Copy move constuctor + /// Copy move constructor /// /// @param vmproxy The source proxy ProtoVolumeMaterial(ProtoVolumeMaterial&& vmproxy) = default; diff --git a/Core/include/Acts/Material/VolumeMaterialMapper.hpp b/Core/include/Acts/Material/VolumeMaterialMapper.hpp index b8f877324f6..4354d68a60b 100644 --- a/Core/include/Acts/Material/VolumeMaterialMapper.hpp +++ b/Core/include/Acts/Material/VolumeMaterialMapper.hpp @@ -80,7 +80,7 @@ class VolumeMaterialMapper { /// /// Nested State struct which is used for the mapping prococess struct State { - /// Constructor of the Sate with contexts + /// Constructor of the State with contexts State(const GeometryContext& gctx, const MagneticFieldContext& mctx) : geoContext(gctx), magFieldContext(mctx) {} diff --git a/Core/include/Acts/Material/detail/AverageMaterials.hpp b/Core/include/Acts/Material/detail/AverageMaterials.hpp index ef4c845907e..dfabeb61de7 100644 --- a/Core/include/Acts/Material/detail/AverageMaterials.hpp +++ b/Core/include/Acts/Material/detail/AverageMaterials.hpp @@ -20,7 +20,7 @@ namespace detail { /// @returns Material slab with the combined thickness and average parameters /// /// The averaged material slab has the combined thickness of the two input slabs -/// and assumes the two input materials are homogeneously and continously mixed +/// and assumes the two input materials are homogeneously and continuously mixed /// throughout the slab. MaterialSlab combineSlabs(const MaterialSlab& slab1, const MaterialSlab& slab2); diff --git a/Core/include/Acts/Navigation/DetectorVolumeUpdators.hpp b/Core/include/Acts/Navigation/DetectorVolumeUpdators.hpp index b4e6ae46404..5b4343311b7 100644 --- a/Core/include/Acts/Navigation/DetectorVolumeUpdators.hpp +++ b/Core/include/Acts/Navigation/DetectorVolumeUpdators.hpp @@ -27,7 +27,7 @@ class DetectorVolume; /// navigation state to nullptr, usually indicates the end of /// the known world, hence the name struct EndOfWorldImpl : public INavigationDelegate { - /// @brief a null volume link - explicitely + /// @brief a null volume link - explicitly /// /// @note the method parameters are ignored inline void update(const GeometryContext& /*ignored*/, @@ -54,7 +54,7 @@ struct SingleDetectorVolumeImpl : public INavigationDelegate { SingleDetectorVolumeImpl() = delete; - /// @brief a null volume link - explicitely + /// @brief a null volume link - explicitly /// /// @note the method parameters are ignored /// diff --git a/Core/include/Acts/Navigation/NavigationState.hpp b/Core/include/Acts/Navigation/NavigationState.hpp index 7a8682ec309..0e924004ca0 100644 --- a/Core/include/Acts/Navigation/NavigationState.hpp +++ b/Core/include/Acts/Navigation/NavigationState.hpp @@ -88,8 +88,8 @@ struct NavigationState { /// An overstep tolerance ActsScalar overstepTolerance = -0.1; - /// Auxilliary attached information - std::any auxilliary; + /// Auxiliary attached information + std::any auxiliary; }; } // namespace Experimental diff --git a/Core/include/Acts/Navigation/NextNavigator.hpp b/Core/include/Acts/Navigation/NextNavigator.hpp index b0beb5268ba..acad5308ee3 100644 --- a/Core/include/Acts/Navigation/NextNavigator.hpp +++ b/Core/include/Acts/Navigation/NextNavigator.hpp @@ -163,7 +163,7 @@ class NextNavigator { /// @brief Navigator pre step call /// - /// This will invalide the current surface and current portal in order + /// This will invalid the current surface and current portal in order /// to navigate to the next ones. /// /// @tparam propagator_state_t is the type of Propagatgor state diff --git a/Core/include/Acts/Navigation/SurfaceCandidatesUpdators.hpp b/Core/include/Acts/Navigation/SurfaceCandidatesUpdators.hpp index b14085885c6..b644d7c1a42 100644 --- a/Core/include/Acts/Navigation/SurfaceCandidatesUpdators.hpp +++ b/Core/include/Acts/Navigation/SurfaceCandidatesUpdators.hpp @@ -40,7 +40,7 @@ inline static void updateCandidates(const GeometryContext& gctx, auto& nCandidates = nState.surfaceCandidates; for (auto& c : nCandidates) { - // Get the surface reprensentation: either native surfcae of portal + // Get the surface representation: either native surfcae of portal const Surface& sRep = (c.surface != nullptr) ? (*c.surface) : (c.portal->surface()); @@ -191,7 +191,7 @@ template using IndexedSurfacesImpl = IndexedUpdatorImpl; -/// @brief An indexed surface implementaion with portal access +/// @brief An indexed surface implementation with portal access template using IndexedSurfacesAllPortalsImpl = ChainedUpdatorImpl, AllPortalsImpl>; diff --git a/Core/include/Acts/Propagator/AbortList.hpp b/Core/include/Acts/Propagator/AbortList.hpp index cce062976d3..d55e317f493 100644 --- a/Core/include/Acts/Propagator/AbortList.hpp +++ b/Core/include/Acts/Propagator/AbortList.hpp @@ -92,7 +92,7 @@ struct AbortList : public detail::Extendable { } /// This is the call signature for the abort list, it broadcasts the call - /// to the tuple() memembers of the list + /// to the tuple() members of the list /// /// @tparam propagator_state_t is the state type of the propagator /// @tparam stepper_t Type of the stepper diff --git a/Core/include/Acts/Propagator/ActionList.hpp b/Core/include/Acts/Propagator/ActionList.hpp index 1e0a4fee8a0..fce63d90617 100644 --- a/Core/include/Acts/Propagator/ActionList.hpp +++ b/Core/include/Acts/Propagator/ActionList.hpp @@ -37,7 +37,7 @@ struct ActionList : public detail::Extendable { using detail::Extendable::tuple; public: - // This uses the type collector and unpacks using the `R` meta funciton + // This uses the type collector and unpacks using the `R` meta function template