Skip to content

Commit beb8854

Browse files
committed
Fix linter errors
1 parent 5492d1c commit beb8854

File tree

3 files changed

+22
-11
lines changed

3 files changed

+22
-11
lines changed

Common/TableProducer/trackPropagationTester.cxx

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,10 @@
99
// granted to it by virtue of its status as an Intergovernmental Organization
1010
// or submit itself to any jurisdiction.
1111

12+
/// \file trackPropagationTester.cxx
13+
/// \brief testing ground for track propagation
14+
/// \author ALICE
15+
1216
//===============================================================
1317
//
1418
// Experimental version of the track propagation task
@@ -52,9 +56,9 @@ using namespace o2::framework;
5256
// using namespace o2::framework::expressions;
5357

5458
struct TrackPropagationTester {
55-
o2::common::standardCCDBLoaderConfigurables standardCCDBLoaderConfigurables;
56-
o2::common::trackPropagationProducts trackPropagationProducts;
57-
o2::common::trackPropagationConfigurables trackPropagationConfigurables;
59+
o2::common::StandardCCDBLoaderConfigurables standardCCDBLoaderConfigurables;
60+
o2::common::TrackPropagationProducts trackPropagationProducts;
61+
o2::common::TrackPropagationConfigurables trackPropagationConfigurables;
5862

5963
Service<o2::ccdb::BasicCCDBManager> ccdb;
6064

Common/Tools/StandardCCDBLoader.h

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,10 @@
99
// granted to it by virtue of its status as an Intergovernmental Organization
1010
// or submit itself to any jurisdiction.
1111

12+
/// \file StandardCCDBLoader.cxx
13+
/// \brief A simple object to handle ccdb queries
14+
/// \author ALICE
15+
1216
#ifndef COMMON_TOOLS_STANDARDCCDBLOADER_H_
1317
#define COMMON_TOOLS_STANDARDCCDBLOADER_H_
1418

@@ -28,11 +32,10 @@ namespace common
2832
{
2933

3034
// ConfigurableGroup with locations
31-
struct standardCCDBLoaderConfigurables : o2::framework::ConfigurableGroup {
35+
struct StandardCCDBLoaderConfigurables : o2::framework::ConfigurableGroup {
3236
std::string prefix = "ccdb";
3337
o2::framework::Configurable<std::string> ccdburl{"ccdb-url", "http://alice-ccdb.cern.ch", "url of the ccdb repository"};
3438
o2::framework::Configurable<std::string> lutPath{"lutPath", "GLO/Param/MatLUT", "Path of the Lut parametrization"};
35-
o2::framework::Configurable<std::string> geoPath{"geoPath", "GLO/Config/GeometryAligned", "Path of the geometry file"};
3639
o2::framework::Configurable<std::string> grpmagPath{"grpmagPath", "GLO/Config/GRPMagField", "CCDB path of the GRPMagField object"};
3740
o2::framework::Configurable<std::string> mVtxPath{"mVtxPath", "GLO/Calib/MeanVertex", "Path of the mean vertex file"};
3841
};

Common/Tools/TrackPropagationModule.h

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,12 @@
99
// granted to it by virtue of its status as an Intergovernmental Organization
1010
// or submit itself to any jurisdiction.
1111

12-
#ifndef COMMON_TOOLS_TRACKPROPAGATIONFUNCTIONS_H_
13-
#define COMMON_TOOLS_TRACKPROPAGATIONFUNCTIONS_H_
12+
/// \file TrackPropagationModule.h
13+
/// \brief track propagation module functionality to be used in tasks
14+
/// \author ALICE
15+
16+
#ifndef COMMON_TOOLS_TRACKPROPAGATIONMODULE_H_
17+
#define COMMON_TOOLS_TRACKPROPAGATIONMODULE_H_
1418

1519
#include <cstdlib>
1620
#include <cmath>
@@ -36,7 +40,7 @@ namespace o2
3640
namespace common
3741
{
3842

39-
struct trackPropagationProducts : o2::framework::ProducesGroup {
43+
struct TrackPropagationProducts : o2::framework::ProducesGroup {
4044
o2::framework::Produces<aod::StoredTracks> tracksParPropagated;
4145
o2::framework::Produces<aod::TracksExtension> tracksParExtensionPropagated;
4246
o2::framework::Produces<aod::StoredTracksCov> tracksParCovPropagated;
@@ -46,7 +50,7 @@ struct trackPropagationProducts : o2::framework::ProducesGroup {
4650
o2::framework::Produces<aod::TrackTunerTable> tunertable;
4751
};
4852

49-
struct trackPropagationConfigurables : o2::framework::ConfigurableGroup {
53+
struct TrackPropagationConfigurables : o2::framework::ConfigurableGroup {
5054
std::string prefix = "trackPropagation";
5155
o2::framework::Configurable<float> minPropagationRadius{"minPropagationDistance", o2::constants::geom::XTPCInnerRef + 0.1, "Only tracks which are at a smaller radius will be propagated, defaults to TPC inner wall"};
5256
// for TrackTuner only (MC smearing)
@@ -136,7 +140,7 @@ class TrackPropagationModule
136140
}
137141
}
138142

139-
for (auto& track : tracks) {
143+
for (const auto& track : tracks) {
140144
if (fillTracksCov) {
141145
if (fillTracksDCA || fillTracksDCACov) {
142146
mDcaInfoCov.set(999, 999, 999, 999, 999);
@@ -250,4 +254,4 @@ class TrackPropagationModule
250254
} // namespace common
251255
} // namespace o2
252256

253-
#endif // COMMON_TOOLS_TRACKPROPAGATIONFUNCTIONS_H_
257+
#endif // COMMON_TOOLS_TRACKPROPAGATIONMODULE_H_

0 commit comments

Comments
 (0)