Skip to content

[PWGDQ] Update tableMaker_withAssoc.cxx#8312

Merged
iarsene merged 12 commits into
AliceO2Group:masterfrom
shreyasiacharya:master
Nov 5, 2024
Merged

[PWGDQ] Update tableMaker_withAssoc.cxx#8312
iarsene merged 12 commits into
AliceO2Group:masterfrom
shreyasiacharya:master

Conversation

@shreyasiacharya

Copy link
Copy Markdown
Contributor
  1. Added a histogram to determine the percentage of orphan tracks.
  2. Now, we store the original event ID in the "trackBasic" table instead of the first amongst the associated collisions. This is necessary for calculation of the proper delta-time (track time - collision time with BC offset), example when using the TableReader_withAssoc

1. Added a histogram to determine the percentage of orphan tracks.
2. Now, we store the original event ID in the "trackBasic" table instead of the first amongst the associated collisions. This is necessary for calculation of the proper delta-time (track time - collision time with BC offset), example when using the TableReader_withAssoc
@github-actions github-actions Bot added the pwgdq label Nov 5, 2024
@github-actions github-actions Bot changed the title Update tableMaker_withAssoc.cxx [PWGDQ] Update tableMaker_withAssoc.cxx Nov 5, 2024
@alibuild

alibuild commented Nov 5, 2024

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for 3413ac3 at 2024-11-05 15:25:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/8312-slc7_x86-64/0/PWGDQ/TableProducer/tableMaker_withAssoc.cxx:613:30: error: 'tracks' was not declared in this scope; did you mean 'track'?
ninja: build stopped: subcommand failed.

Full log here.

@alibuild

alibuild commented Nov 5, 2024

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for ac2c55d at 2024-11-05 16:41:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/8312-slc7_x86-64/0/PWGDQ/TableProducer/tableMaker_withAssoc.cxx:613:30: error: 'tracks' was not declared in this scope; did you mean 'track'?
ninja: build stopped: subcommand failed.

Full log here.

@iarsene iarsene enabled auto-merge (squash) November 5, 2024 15:44
@iarsene iarsene merged commit 414e441 into AliceO2Group:master Nov 5, 2024
@vkucera

vkucera commented Nov 6, 2024

Copy link
Copy Markdown
Collaborator

@shreyasiacharya Many errors reported by cpplint.

@iarsene

iarsene commented Nov 6, 2024

Copy link
Copy Markdown
Collaborator

@vkucera We will try to fix them in a separate commit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

4 participants