Skip to content

Label renamed to Lbl, MC information in dumpDataModel#3603

Merged
pzhristov merged 1 commit intoAliceO2Group:devfrom
pzhristov:dev
May 18, 2020
Merged

Label renamed to Lbl, MC information in dumpDataModel#3603
pzhristov merged 1 commit intoAliceO2Group:devfrom
pzhristov:dev

Conversation

@pzhristov
Copy link
Copy Markdown
Contributor

Rename Label to Lbl since label in O2 is used in different scope.
Add the MC information to dumpDataModel

@pzhristov pzhristov requested review from a team and jgrosseo as code owners May 17, 2020 19:20
Copy link
Copy Markdown
Collaborator

@jgrosseo jgrosseo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In a next PR, we will consider

  • Using Label if possible after soa changes
  • Rename MCLabel table to MCTrackLabels
  • Move negative values to bits and use DECLARE_SOA_INDEX_COLUMN(MCTracks, ...)

@pzhristov pzhristov merged commit 9c7da2e into AliceO2Group:dev May 18, 2020
@pzhristov
Copy link
Copy Markdown
Contributor Author

pzhristov commented May 18, 2020

Thanks @jgrosseo, I am preparing the next pull request as discussed.

amorsch pushed a commit to amorsch/AliceO2 that referenced this pull request May 28, 2020
… the MC information to dumpDataModel (AliceO2Group#3603)

Co-authored-by: hristov <Peter.Hristov@cern.ch>
EmilGorm pushed a commit to EmilGorm/AliceO2 that referenced this pull request Nov 22, 2021
… the MC information to dumpDataModel (AliceO2Group#3603)

Co-authored-by: hristov <Peter.Hristov@cern.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants