[Common] Fix includes and using statements

Common/CCDB/EventSelectionParams.cxx

@@ -16,7 +16,7 @@
 
 // o2-linter: disable=name/workflow-file
 
-#include "EventSelectionParams.h"
+#include "Common/CCDB/EventSelectionParams.h"
 
 #include <cstddef>
 #include <cstring>

Common/CCDB/EventSelectionParams.h

@@ -17,9 +17,10 @@
 #ifndef COMMON_CCDB_EVENTSELECTIONPARAMS_H_
 #define COMMON_CCDB_EVENTSELECTIONPARAMS_H_
 
-#include <Rtypes.h>
 #include <TMath.h>
 
+#include <Rtypes.h>
+
 namespace o2::aod::evsel
 {
 // Event selection criteria

Common/CCDB/macros/upload_event_selection_params.C

@@ -9,12 +9,16 @@
 // granted to it by virtue of its status as an Intergovernmental Organization
 // or submit itself to any jurisdiction.
 
-#include "CCDB/CcdbApi.h"
+#include "Common/CCDB/EventSelectionParams.h"
+
 #include "CCDB/BasicCCDBManager.h"
+#include "CCDB/CcdbApi.h"
+
 #include "TString.h"
+
 #include <map>
 #include <string>
-#include "EventSelectionParams.h"
+
 using std::map;
 using std::string;
 

Common/CCDB/macros/upload_event_selection_params_run3.C

@@ -9,11 +9,18 @@
 // granted to it by virtue of its status as an Intergovernmental Organization
 // or submit itself to any jurisdiction.
 
+#include "Common/CCDB/EventSelectionParams.h"
+
 #include "CCDB/CcdbApi.h"
+#include <Framework/Logger.h>
+
 #include "TString.h"
+
+#include <RtypesCore.h>
+
+#include <cstdlib>
 #include <map>
 #include <string>
-#include "EventSelectionParams.h"
 using std::map;
 using std::string;
 
@@ -140,4 +147,4 @@ void fillMapOfCustomOrbitShifts(std::map<int, int>& mapOrbitShift)
   mapOrbitShift[523797] = 41;
   mapOrbitShift[523821] = 36;
   mapOrbitShift[523897] = 38;
-}
+}

Common/CCDB/macros/upload_trigger_aliases.C

@@ -9,12 +9,17 @@
 // granted to it by virtue of its status as an Intergovernmental Organization
 // or submit itself to any jurisdiction.
 
-#include "CCDB/CcdbApi.h"
+#include "Common/CCDB/TriggerAliases.h"
+
 #include "CCDB/BasicCCDBManager.h"
+#include "CCDB/CcdbApi.h"
+#include <Framework/Logger.h>
+
 #include "TObjArray.h"
-#include "TriggerAliases.h"
-#include "TTree.h"
 #include "TString.h"
+
+#include <RtypesCore.h>
+
 #include <fstream>
 #include <map>
 #include <string>

Common/CCDB/macros/upload_trigger_aliases_run3.C

@@ -9,13 +9,18 @@
 // granted to it by virtue of its status as an Intergovernmental Organization
 // or submit itself to any jurisdiction.
 
-#include "DataFormatsCTP/Configuration.h"
-#include "CCDB/CcdbApi.h"
+#include "Common/CCDB/TriggerAliases.h"
+
 #include "CCDB/BasicCCDBManager.h"
+#include "CCDB/CcdbApi.h"
+#include "DataFormatsCTP/Configuration.h"
+#include <Framework/Logger.h>
+
 #include "TObjArray.h"
-#include "TriggerAliases.h"
-#include "TTree.h"
 #include "TString.h"
+
+#include <RtypesCore.h>
+
 #include <fstream>
 #include <map>
 #include <string>

Common/Core/EventMixing.h

@@ -9,8 +9,8 @@
 // granted to it by virtue of its status as an Intergovernmental Organization
 // or submit itself to any jurisdiction.
 
-#ifndef ANALYSIS_CORE_EVENTMIXING_H_
-#define ANALYSIS_CORE_EVENTMIXING_H_
+#ifndef COMMON_CORE_EVENTMIXING_H_
+#define COMMON_CORE_EVENTMIXING_H_
 
 namespace eventmixing
 {
@@ -47,4 +47,4 @@ static int getMixingBin(const T1& vtxBins, const T1& multBins, const T2& vtx, co
 }
 }; // namespace eventmixing
 
-#endif /* ANALYSIS_CORE_EVENTMIXING_H_ */
+#endif // COMMON_CORE_EVENTMIXING_H_

Common/Core/FFitWeights.cxx

@@ -16,12 +16,14 @@
 
 #include "FFitWeights.h"
 
-#include "Framework/Logger.h"
+#include <Framework/Logger.h>
 
 #include <TCollection.h>
 #include <TH1.h>
+#include <TH2.h>
 #include <TNamed.h>
 #include <TObjArray.h>
+#include <TProfile.h>
 #include <TSpline.h>
 #include <TString.h>
 

Common/Core/MetadataHelper.cxx

@@ -21,6 +21,7 @@
 #include <Framework/InitContext.h>
 #include <Framework/Logger.h>
 
+#include <TString.h>
 #include <TSystem.h>
 
 #include <array>

Common/Core/PID/PIDTOFParamService.cxx

@@ -18,11 +18,24 @@
 
 #include "PIDTOFParamService.h"
 
+#include "Common/Core/CollisionTypeHelper.h"
+#include "Common/Core/MetadataHelper.h"
+
+#include <CCDB/BasicCCDBManager.h>
+#include <DataFormatsParameters/GRPLHCIFData.h>
+#include <DataFormatsTOF/ParameterContainers.h>
 #include <Framework/CommonServices.h>
+#include <Framework/Logger.h>
 #include <Framework/Plugins.h>
 #include <Framework/ServiceHandle.h>
 #include <Framework/ServiceSpec.h>
+#include <Framework/TypeIdHelpers.h>
+#include <PID/PIDTOF.h>
+
+#include <TGraph.h>
 
+#include <chrono>
+#include <cstdint>
 #include <map>
 #include <string>
 

Common/Core/PID/PIDTOFParamService.h

@@ -24,14 +24,16 @@
 #include "Common/Core/PID/PIDTOF.h"
 #include "Common/Core/TableHelper.h"
 
-#include "CCDB/BasicCCDBManager.h"
-#include "CommonConstants/PhysicsConstants.h"
-#include "DataFormatsTOF/ParameterContainers.h"
-#include "Framework/DataTypes.h"
-#include "Framework/PID.h"
-#include "Framework/Plugins.h"
-#include "ReconstructionDataFormats/PID.h"
-
+#include <CCDB/BasicCCDBManager.h>
+#include <CommonConstants/PhysicsConstants.h>
+#include <Framework/InitContext.h>
+#include <Framework/Logger.h>
+#include <Framework/PID.h>
+#include <Framework/Plugins.h>
+#include <ReconstructionDataFormats/PID.h>
+
+#include <cmath>
+#include <cstdint>
 #include <string>
 
 namespace o2::pid::tof
@@ -188,7 +190,7 @@ struct TOFResponseImpl {
   template <typename VType>
   void getCfg(o2::framework::InitContext& initContext, const std::string name, VType& v, const std::string task)
   {
-    if (!getTaskOptionValue(initContext, task, name, v, false)) {
+    if (!o2::common::core::getTaskOptionValue(initContext, task, name, v, false)) {
       LOG(fatal) << "Could not get " << name << " from " << task << " task";
     }
   }

Common/Core/PID/TPCPIDResponse.h

@@ -17,8 +17,8 @@
 #ifndef COMMON_CORE_PID_TPCPIDRESPONSE_H_
 #define COMMON_CORE_PID_TPCPIDRESPONSE_H_
 
-#include <MathUtils/BetheBlochAleph.h>
 #include <Framework/Logger.h>
+#include <MathUtils/BetheBlochAleph.h>
 #include <ReconstructionDataFormats/PID.h>
 
 #include <Rtypes.h>

Common/Core/TableHelper.h

@@ -121,9 +121,4 @@ bool getTaskOptionValue(o2::framework::InitContext& initContext, const std::stri
 
 } // namespace o2::common::core
 
-using o2::common::core::enableFlagIfTableRequired;
-using o2::common::core::getTaskOptionValue;
-using o2::common::core::isTableRequiredInWorkflow;
-using o2::common::core::printTablesInWorkflow;
-
 #endif // COMMON_CORE_TABLEHELPER_H_

Common/Core/Zorro.cxx

@@ -33,7 +33,6 @@
 #include <bitset>
 #include <cstddef>
 #include <cstdint>
-#include <map>
 #include <memory>
 #include <string>
 #include <vector>

Common/Core/Zorro.h

@@ -16,8 +16,8 @@
 /// \author M Puccio
 ///
 
-#ifndef EVENTFILTERING_ZORRO_H_
-#define EVENTFILTERING_ZORRO_H_
+#ifndef COMMON_CORE_ZORRO_H_
+#define COMMON_CORE_ZORRO_H_
 
 #include "ZorroHelper.h"
 #include "ZorroSummary.h"
@@ -103,4 +103,4 @@ class Zorro
   o2::ccdb::BasicCCDBManager* mCCDB = nullptr;
 };
 
-#endif // EVENTFILTERING_ZORRO_H_
+#endif // COMMON_CORE_ZORRO_H_

Common/Core/ZorroHelper.h

@@ -10,8 +10,8 @@
 // or submit itself to any jurisdiction.
 //
 
-#ifndef EVENTFILTERING_ZORROHELPER_H_
-#define EVENTFILTERING_ZORROHELPER_H_
+#ifndef COMMON_CORE_ZORROHELPER_H_
+#define COMMON_CORE_ZORROHELPER_H_
 
 #include <Rtypes.h>
 #include <RtypesCore.h>
@@ -21,4 +21,4 @@ struct ZorroHelper {
   ClassDefNV(ZorroHelper, 1);
 };
 
-#endif // EVENTFILTERING_ZORROHELPER_H_
+#endif // COMMON_CORE_ZORROHELPER_H_

Common/Core/ZorroSummary.h

@@ -10,14 +10,15 @@
 // or submit itself to any jurisdiction.
 //
 
-#ifndef EVENTFILTERING_ZORROSUMMARY_H_
-#define EVENTFILTERING_ZORROSUMMARY_H_
+#ifndef COMMON_CORE_ZORROSUMMARY_H_
+#define COMMON_CORE_ZORROSUMMARY_H_
 
 #include <TNamed.h>
 
 #include <Rtypes.h>
 #include <RtypesCore.h>
 
+#include <cstddef>
 #include <string>
 #include <unordered_map>
 #include <vector>
@@ -80,4 +81,4 @@ class ZorroSummary : public TNamed
   ClassDef(ZorroSummary, 1);
 };
 
-#endif // EVENTFILTERING_ZORROSUMMARY_H_
+#endif // COMMON_CORE_ZORROSUMMARY_H_

Common/Core/fwdtrackUtilities.h

@@ -18,18 +18,18 @@
 #ifndef COMMON_CORE_FWDTRACKUTILITIES_H_
 #define COMMON_CORE_FWDTRACKUTILITIES_H_
 
-#include "DetectorsBase/GeometryManager.h"
-#include "Field/MagneticField.h"
-#include "Framework/AnalysisDataModel.h"
-#include "GlobalTracking/MatchGlobalFwd.h"
-#include "MCHTracking/TrackExtrap.h"
-#include "ReconstructionDataFormats/GlobalFwdTrack.h"
-#include "ReconstructionDataFormats/TrackFwd.h"
-
-#include "Math/MatrixRepresentationsStatic.h"
-#include "Math/SMatrix.h"
-#include "TGeoGlobalMagField.h"
-
+#include <Framework/AnalysisDataModel.h>
+#include <Framework/DataTypes.h>
+#include <GlobalTracking/MatchGlobalFwd.h>
+#include <MCHTracking/TrackExtrap.h>
+#include <ReconstructionDataFormats/GlobalFwdTrack.h>
+#include <ReconstructionDataFormats/TrackFwd.h>
+
+#include <Math/MatrixRepresentationsStatic.h>
+#include <Math/SMatrix.h>
+
+#include <array>
+#include <cstdint>
 #include <type_traits>
 #include <vector>
 

Common/DataModel/EventSelection.h

@@ -17,10 +17,6 @@
 #ifndef COMMON_DATAMODEL_EVENTSELECTION_H_
 #define COMMON_DATAMODEL_EVENTSELECTION_H_
 
-#include "Common/CCDB/EventSelectionParams.h" // FIXME: remove
-#include "Common/CCDB/RCTSelectionFlags.h"    // FIXME: remove
-#include "Common/CCDB/TriggerAliases.h"       // FIXME: remove
-
 #include <Framework/AnalysisDataModel.h>
 
 namespace o2::aod

Common/DataModel/PIDResponseITS.h

@@ -142,7 +142,7 @@ struct ITSResponse {
     float p0_res = 0, p1_res = 0, p2_res = 0;
     float p0_res_Z2 = 0, p1_res_Z2 = 0, p2_res_Z2 = 0;
     o2::framework::LabeledArray<float> itsParams;
-    getTaskOptionValue(initContext, "its-pid", "itsParams", itsParams, true);
+    o2::common::core::getTaskOptionValue(initContext, "its-pid", "itsParams", itsParams, true);
     auto data = itsParams.getData();
     const int col = isMC ? 1 : 0; // 0 for Data, 1 for MC
     if (data.rows != 2 || data.cols != 12) {

Common/DataModel/PIDResponseTOF.h

@@ -23,6 +23,7 @@
 #include "Common/Core/PID/PIDTOF.h"
 #include "Common/Core/PID/PIDTOFParamService.h"
 
+#include <CommonConstants/LHCConstants.h>
 #include <Framework/ASoA.h>
 #include <Framework/AnalysisDataModel.h>
 #include <Framework/Logger.h>

Common/DataModel/ZDCLightIons.h

@@ -16,10 +16,9 @@
 #ifndef COMMON_DATAMODEL_ZDCLIGHTIONS_H_
 #define COMMON_DATAMODEL_ZDCLIGHTIONS_H_
 
-#include "Common/DataModel/Centrality.h"
+#include <Framework/AnalysisDataModel.h>
 
-#include "Framework/ASoAHelpers.h"
-#include "Framework/AnalysisDataModel.h"
+#include <cstdint>
 
 namespace o2::aod
 {

Common/TableProducer/Converters/run2TinyToFullPID.cxx

@@ -83,7 +83,7 @@ struct Run2TinyToFullPID {
     for (int i = 0; i < nTables; i++) {
       LOGF(info, "test %i", i);
       int f = enabledTables->get(tableNames[i].c_str(), "enable");
-      enableFlagIfTableRequired(context, tableNames[i], f);
+      o2::common::core::enableFlagIfTableRequired(context, tableNames[i], f);
       if (f == 1) {
         mEnabledTables.push_back(i);
       }