Skip to content

[PWGLF] Please consider the following formatting changes to #11707 #12

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
maciacco merged 1 commit into from
Jun 20, 2025
Merged

[PWGLF] Please consider the following formatting changes to #11707 #12

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
40 changes: 20 additions & 20 deletions PWGLF/TableProducer/Strangeness/strangeTreeCreator.cxx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,33 +13,33 @@
/// \brief table producer for strangeness studies
/// \author Mario Ciacco <mario.ciacco@cern.ch>

#include <vector>
#include <utility>
#include <random>
#include "PWGLF/DataModel/LFSlimStrangeTables.h"
#include "PWGLF/DataModel/LFStrangenessTables.h"

#include "Framework/runDataProcessing.h"
#include "Framework/AnalysisTask.h"
#include "Framework/AnalysisDataModel.h"
#include "Framework/ASoAHelpers.h"
#include "ReconstructionDataFormats/Track.h"
#include "Common/DataModel/TrackSelectionTables.h"
#include "Common/DataModel/Centrality.h"
#include "Common/DataModel/Multiplicity.h"
#include "Common/Core/PID/TPCPIDResponse.h"
#include "Common/Core/RecoDecay.h"
#include "Common/Core/trackUtilities.h"
#include "Common/DataModel/Centrality.h"
#include "Common/DataModel/EventSelection.h"
#include "PWGLF/DataModel/LFStrangenessTables.h"
#include "DetectorsBase/Propagator.h"
#include "DetectorsBase/GeometryManager.h"
#include "DataFormatsParameters/GRPObject.h"
#include "DataFormatsParameters/GRPMagField.h"
#include "CCDB/BasicCCDBManager.h"

#include "Common/Core/PID/TPCPIDResponse.h"
#include "Common/DataModel/Multiplicity.h"
#include "Common/DataModel/PIDResponse.h"
#include "Common/DataModel/TrackSelectionTables.h"

#include "CCDB/BasicCCDBManager.h"
#include "DCAFitter/DCAFitterN.h"
#include "DataFormatsParameters/GRPMagField.h"
#include "DataFormatsParameters/GRPObject.h"
#include "DetectorsBase/GeometryManager.h"
#include "DetectorsBase/Propagator.h"
#include "Framework/ASoAHelpers.h"
#include "Framework/AnalysisDataModel.h"
#include "Framework/AnalysisTask.h"
#include "Framework/runDataProcessing.h"
#include "ReconstructionDataFormats/Track.h"

#include "PWGLF/DataModel/LFSlimStrangeTables.h"
#include <random>
#include <utility>
#include <vector>

using namespace o2;
using namespace o2::framework;
Expand Down Expand Up @@ -180,9 +180,9 @@
Configurable<int> cfgMaterialCorrection{"cfgMaterialCorrection", static_cast<int>(o2::base::Propagator::MatCorrType::USEMatCorrLUT), "Type of material correction"};

ConfigurableAxis centAxis{"centAxis", {106, 0, 106}, "binning for the centrality"};
ConfigurableAxis zVtxAxis{"zVtxBins", {100, -20.f, 20.f}, "Binning for the vertex z in cm"};

Check failure on line 183 in PWGLF/TableProducer/Strangeness/strangeTreeCreator.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[name/configurable] 

Use lowerCamelCase for names of configurables and use the same name for the struct member as for the JSON string. (Declare the type and names on the same line.)
ConfigurableAxis etaAxis{"etaAxis", {8, -0.8f, 0.8f}, "binning for pseudorapidity"};
ConfigurableAxis massKineAxis{"kineAxis", {3000, -3.f, 3.f}, "binning for the kinematic-transofrmed mass shift distributions"};

Check failure on line 185 in PWGLF/TableProducer/Strangeness/strangeTreeCreator.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[name/configurable] 

Use lowerCamelCase for names of configurables and use the same name for the struct member as for the JSON string. (Declare the type and names on the same line.)

// binning of (anti)lambda mass QA histograms
ConfigurableAxis massLambdaAxis{"massLambdaAxis", {400, o2::constants::physics::MassLambda0 - 0.03f, o2::constants::physics::MassLambda0 + 0.03f}, "binning for the lambda invariant-mass"};
Expand All @@ -192,7 +192,7 @@
Configurable<float> zVtxMax{"zVtxMax", 10.0f, "maximum z position of the primary vertex"};
Configurable<float> etaMax{"etaMax", 0.8f, "maximum eta"};
Configurable<float> etaMaxV0dau{"etaMaxV0dau", 0.8f, "maximum eta V0 daughters"};
ConfigurableAxis momAxis{"momAxisFine", {5.e2, 0.f, 5.f}, "momentum axis binning"};

Check failure on line 195 in PWGLF/TableProducer/Strangeness/strangeTreeCreator.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[name/configurable] 

Use lowerCamelCase for names of configurables and use the same name for the struct member as for the JSON string. (Declare the type and names on the same line.)

Configurable<float> downscaleFactor{"downscaleFactor", 1.f, "downscaling factor"};
Configurable<bool> applyAdditionalEvSel{"applyAdditionalEvSel", false, "apply additional event selections"};
Expand Down Expand Up @@ -554,7 +554,7 @@
{
fillRecoEvent<C, T>(collision, tracks, V0s, V0s_all, cascades, centrality);

for (auto& candidateV0 : candidateV0s) { // o2-linter disable=const-red-in-for-loops (non const)

Check failure on line 557 in PWGLF/TableProducer/Strangeness/strangeTreeCreator.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[const-ref-in-for-loop] 

Use constant references for non-modified iterators in range-based for loops.
candidateV0.isreco = true;
auto mcLabPos = mcLabels.rawIteratorAt(candidateV0.globalIndexPos);
auto mcLabNeg = mcLabels.rawIteratorAt(candidateV0.globalIndexNeg);
Expand Down Expand Up @@ -691,7 +691,7 @@
std::array<float, 3> momNegMC = std::array{static_cast<float>(-999.), static_cast<float>(-999.), static_cast<float>(-999.)};

// look for lambda or k0short
int pdg_test = PDG_t::kLambda0;

Check failure on line 694 in PWGLF/TableProducer/Strangeness/strangeTreeCreator.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[name/function-variable] 

Use lowerCamelCase for names of functions and variables.
if (k0short)
pdg_test = PDG_t::kK0Short;

Expand All @@ -699,7 +699,7 @@
if (!mcPart.isPhysicalPrimary() && !mcPart.has_mothers())
continue;
// check if its the right decay containing proton for lambda and charged pion for k0short
int pdg_particle;

Check failure on line 702 in PWGLF/TableProducer/Strangeness/strangeTreeCreator.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[name/function-variable] 

Use lowerCamelCase for names of functions and variables.
if (k0short) {
pdg_particle = PDG_t::kPiPlus;
} else {
Expand Down Expand Up @@ -789,8 +789,8 @@
histos.fill(HIST("QA/zVtx"), collision.posZ());

const uint64_t collIdx = collision.globalIndex();
auto V0TableThisCollision = V0s.sliceBy(perCollisionV0, collIdx);

Check failure on line 792 in PWGLF/TableProducer/Strangeness/strangeTreeCreator.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[name/function-variable] 

Use lowerCamelCase for names of functions and variables.
auto CascTableThisCollision = cascades.sliceBy(perCollisionCasc, collIdx);

Check failure on line 793 in PWGLF/TableProducer/Strangeness/strangeTreeCreator.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[name/function-variable] 

Use lowerCamelCase for names of functions and variables.
V0TableThisCollision.bindExternalIndices(&tracks);
CascTableThisCollision.bindExternalIndices(&tracks);
CascTableThisCollision.bindExternalIndices(&V0s);
Expand Down Expand Up @@ -856,8 +856,8 @@
histos.fill(HIST("QA/zVtx"), collision.posZ());

const uint64_t collIdx = collision.globalIndex();
auto V0TableThisCollision = V0s.sliceBy(perCollisionV0, collIdx);

Check failure on line 859 in PWGLF/TableProducer/Strangeness/strangeTreeCreator.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[name/function-variable] 

Use lowerCamelCase for names of functions and variables.
auto CascTableThisCollision = cascades.sliceBy(perCollisionCasc, collIdx);

Check failure on line 860 in PWGLF/TableProducer/Strangeness/strangeTreeCreator.cxx

View workflow job for this annotation

GitHub Actions / O2 linter

[name/function-variable] 

Use lowerCamelCase for names of functions and variables.
V0TableThisCollision.bindExternalIndices(&tracks);
CascTableThisCollision.bindExternalIndices(&tracks);
CascTableThisCollision.bindExternalIndices(&V0s);
Expand Down
Loading