Skip to content

[MCH] modify (pre)clustering workflows to process several ROF per TF#5463

Merged
aphecetche merged 2 commits intoAliceO2Group:devfrom
pillot:clustering
Feb 16, 2021
Merged

[MCH] modify (pre)clustering workflows to process several ROF per TF#5463
aphecetche merged 2 commits intoAliceO2Group:devfrom
pillot:clustering

Conversation

@pillot
Copy link
Collaborator

@pillot pillot commented Feb 12, 2021

These modifications allow to send and process several events per time frame. Each individual interaction in the time frame is identified by an MCH ROF record, which contains the interaction record and the location of the associated objects (e.g. clusters) in the overall list.

@aphecetche @aferrero2707 I created a new PR because I messed-up the draft while trying to solve a conflict.

As we discussed, the modification of the preclustering workflow will temporarily break the linkup with the raw-to-digit workflow. My understanding is that we can go ahead with this PR and the linkup will fixed later on with the introduction of the temporal preclustering.

@pillot pillot requested a review from aphecetche as a code owner February 12, 2021 17:44
@aferrero2707
Copy link
Collaborator

aferrero2707 commented Feb 13, 2021

@pillot yes, the ROF record introduced here is basically equivalent to the time-based clusters that are being introduced in #3644.
We will modify the time clustering so that it directly generates ROF records along with time-ordered vector of digits.

@aphecetche aphecetche merged commit 8d63de6 into AliceO2Group:dev Feb 16, 2021
@pillot pillot deleted the clustering branch February 16, 2021 18:30
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.

3 participants