Skip to content

Please consider the following formatting changes to #12655 #19

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
gmantzar merged 1 commit into from
Aug 20, 2025
Merged

Please consider the following formatting changes to #12655 #19

Show file tree
Hide file tree
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
2 changes: 1 addition & 1 deletion PWGCF/FemtoDream/Core/femtoDreamResoSelectionKStar.h
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// Copyright 2019-2025 CERN and copyright holders of ALICE O2.

Check failure on line 1 in PWGCF/FemtoDream/Core/femtoDreamResoSelectionKStar.h

View workflow job for this annotation

GitHub Actions / O2 linter

[doc/file] 

Provide mandatory file documentation.
// See https://alice-o2.web.cern.ch/copyright for details of the copyright holders.
// All rights not expressly granted are reserved.
//
Expand All @@ -13,7 +13,7 @@
/// \brief Definition of the FemtoDreamResoSelection
/// \author Valentina Mantovani Sarti, TU München valentina.mantovani-sarti@tum.de
/// \author Andi Mathis, TU München, andreas.mathis@ph.tum.de
/// \author Luca Barioglio, TU München, luca.barioglio@cern.ch

Check failure on line 16 in PWGCF/FemtoDream/Core/femtoDreamResoSelectionKStar.h

View workflow job for this annotation

GitHub Actions / O2 linter

[doc/file] 

Documentation for \file is missing, incorrect or misplaced.

#ifndef PWGCF_FEMTODREAM_CORE_FEMTODREAMRESOSELECTIONKSTAR_H_
#define PWGCF_FEMTODREAM_CORE_FEMTODREAMRESOSELECTIONKSTAR_H_
Expand All @@ -34,13 +34,13 @@
#include <array>
#include <cstdint>
#include <fstream>
#include <iostream>

Check failure on line 37 in PWGCF/FemtoDream/Core/femtoDreamResoSelectionKStar.h

View workflow job for this annotation

GitHub Actions / O2 linter

[include-iostream] 

Do not include iostream. Use O2 logging instead.
#include <string>
#include <vector>
#include <utility>
#include <vector>

using namespace o2;

Check failure on line 42 in PWGCF/FemtoDream/Core/femtoDreamResoSelectionKStar.h

View workflow job for this annotation

GitHub Actions / O2 linter

[using-directive] 

Do not put using directives at global scope in headers.
using namespace o2::framework;

Check failure on line 43 in PWGCF/FemtoDream/Core/femtoDreamResoSelectionKStar.h

View workflow job for this annotation

GitHub Actions / O2 linter

[using-directive] 

Do not put using directives at global scope in headers.

namespace o2::analysis::femtoDream
{
Expand Down Expand Up @@ -97,10 +97,10 @@
void setDaughterPIDSpecies(T const& daugh, V& pids);

template <typename V>
bool DaughterSelectionPos(V const& track1, bool UseThreshold);

Check failure on line 100 in PWGCF/FemtoDream/Core/femtoDreamResoSelectionKStar.h

View workflow job for this annotation

GitHub Actions / O2 linter

[name/function-variable] 

Use lowerCamelCase for names of functions and variables.

template <typename V>
bool DaughterSelectionNeg(V const& track2, bool UseThreshold);

Check failure on line 103 in PWGCF/FemtoDream/Core/femtoDreamResoSelectionKStar.h

View workflow job for this annotation

GitHub Actions / O2 linter

[name/function-variable] 

Use lowerCamelCase for names of functions and variables.

template <typename cutContainerType, typename V>
std::array<cutContainerType, 5> getCutContainer(V const& track1, V const& track2, float sign);
Expand Down Expand Up @@ -181,7 +181,7 @@

/// Helper function to obtain the type of a given selection variable for consistent naming of the configurables
/// \param iSel Reso selection variable whose type is returned
static femtoDreamSelection::SelectionType

Check failure on line 184 in PWGCF/FemtoDream/Core/femtoDreamResoSelectionKStar.h

View workflow job for this annotation

GitHub Actions / O2 linter

[name/function-variable] 

Use lowerCamelCase for names of functions and variables.
getSelectionType(femtoDreamResoSelection::ResoSel iSel)
{
return mSelectionTypes[iSel];
Expand All @@ -203,7 +203,7 @@
float mPIDoffsetTPC;
float mPIDoffsetTOF;

FemtoDreamTrackSelection PosDaughTrack;

Check failure on line 206 in PWGCF/FemtoDream/Core/femtoDreamResoSelectionKStar.h

View workflow job for this annotation

GitHub Actions / O2 linter

[name/function-variable] 

Use lowerCamelCase for names of functions and variables.
FemtoDreamTrackSelection NegDaughTrack;

static constexpr int kNresoSelection = 1;
Expand Down Expand Up @@ -384,7 +384,7 @@
{
cutContainerType outputSign = 0;
size_t counter = 0;
for (auto& sel : mSelections) { /// it should just be a 1D vector with sign

Check failure on line 387 in PWGCF/FemtoDream/Core/femtoDreamResoSelectionKStar.h

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.
const auto selVariable = sel.getSelectionVariable();
if (selVariable == femtoDreamResoSelection::kResoSign) {
sel.checkSelectionSetBit(sign, outputSign, counter, nullptr);
Expand Down
2 changes: 1 addition & 1 deletion PWGCF/FemtoDream/Tasks/femtoDreamDebugReso.cxx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,8 @@

#include <cstdint>
#include <iostream>
#include <vector>
#include <string>
#include <vector>

using namespace o2;
using namespace o2::analysis::femtoDream;
Expand Down
3 changes: 0 additions & 3 deletions PWGCF/FemtoDream/Tasks/femtoDreamPairTaskV0Reso.cxx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,15 +9,12 @@
// granted to it by virtue of its status as an Intergovernmental Organization
// or submit itself to any jurisdiction.



/// \file femtoDreamPairTaskV0Reso.cxx
/// \brief Tasks that reads the track tables used for the pairing and builds pairs of two tracks
/// \author Christopher Klumm, TU München, christopher.klumm@cern.ch
/// \author Anton Riedel, TU München, anton.riedel@cern.ch
/// \author Georgios Mantzaridis, TU München, georgios.mantzaridis@cern.ch


#include "PWGCF/DataModel/FemtoDerived.h"
#include "PWGCF/FemtoDream/Core/femtoDreamContainer.h"
#include "PWGCF/FemtoDream/Core/femtoDreamDetaDphiStar.h"
Expand Down
Loading