Skip to content

[Common] Uniform tracks extra converter to the coding conventions #3671

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
ddobrigk merged 2 commits into from
Oct 24, 2023
Merged

[Common] Uniform tracks extra converter to the coding conventions #3671

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
304e0ff
[Common] Uniform tracks extra converter to the coding conventions
fmazzasc Oct 22, 2023
1f0a30f
Add doxygen documentation + capitalize tasks
fmazzasc Oct 23, 2023
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
4 changes: 2 additions & 2 deletions Common/TableProducer/CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -71,8 +71,8 @@ o2physics_add_dpl_workflow(mc-converter
PUBLIC_LINK_LIBRARIES
COMPONENT_NAME Analysis)

o2physics_add_dpl_workflow(tracksextra-converter
SOURCES tracksextraConverter.cxx
o2physics_add_dpl_workflow(tracks-extra-converter
SOURCES tracksExtraConverter.cxx
PUBLIC_LINK_LIBRARIES
COMPONENT_NAME Analysis)

Expand Down
26 changes: 17 additions & 9 deletions ...on/TableProducer/tracksextraConverter.cxx → ...on/TableProducer/tracksExtraConverter.cxx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,22 +8,30 @@
// In applying this license CERN does not waive the privileges and immunities
// granted to it by virtue of its status as an Intergovernmental Organization
// or submit itself to any jurisdiction.

/// \file tracksExtraConverter.cxx
/// \brief Converts TracksExtra table from version 000 to 001

/// This task allows for the conversion of the TracksExtra table from version 000 to 001.
/// The conversion is needed because the table has been extended with the ITSClusterSize column
/// and the ITSClusterMap column is evaluated dynamically from it.
/// In the converter a dummy ITSClusterSize column is filled with overflows if a hit in the layer is present

/// \author F.Mazzaschi <fmazzasc@cern.ch>

#include "Framework/runDataProcessing.h"
#include "Framework/AnalysisTask.h"
#include "Framework/AnalysisDataModel.h"

using namespace o2;
using namespace o2::framework;

// Converts TracksExtra table from version 000 to 001
struct tracksextraConverter {
struct TracksExtraConverter {
Produces<aod::StoredTracksExtra_001> tracksExtra_001;
void process(aod::TracksExtra_000 const& tracksExtra_000)
{

// dummy itsClusterSizes, fill with overflows if a hit in the layer is present

for (auto& track0 : tracksExtra_000) {
for (const auto& track0 : tracksExtra_000) {

uint32_t itsClusterSizes = 0;
for (int layer = 0; layer < 7; layer++) {
Expand Down Expand Up @@ -56,15 +64,15 @@ struct tracksextraConverter {
}
};

struct trackExtraSpawner {
// spawn the extended table for TracksExtra001 to avoid the call to the internal spawner and the circular dependency
/// Spawn the extended table for TracksExtra001 to avoid the call to the internal spawner and a consequent circular dependency
struct TracksExtraSpawner {
Spawns<aod::TracksExtra_001> tracksExtra_001;
};

WorkflowSpec defineDataProcessing(ConfigContext const& cfgc)
{
return WorkflowSpec{
adaptAnalysisTask<tracksextraConverter>(cfgc),
adaptAnalysisTask<trackExtraSpawner>(cfgc),
adaptAnalysisTask<TracksExtraConverter>(cfgc),
adaptAnalysisTask<TracksExtraSpawner>(cfgc),
};
}