Hi @shahor02 @iouribelikov
I compared the effects of the PR before and after the mcTruth usage. In the presented plots there are some tiny improvements, not dramatic, as also expected with this configuration.
I guess then the tagging algorithm seems to work.
Given that the impact is not so high I was considering to postpone the merging and wait for the work of tracking multiple rofs to be merged and adapt everything accordingly.

@mconcas OK. Meanwhile, perhaps you can add the fake patterns flag we have discussed.

@mconcas This PR seems to break the macro/run_trac_alice3.C, see o2-dataflow CI

@mconcas This PR seems to break the macro/run_trac_alice3.C, see o2-dataflow CI

Thanks, I did a quick fix.
This macro will anyways be removed as soon as I make ALICE 3 tracking to work with MCtruth containers.

@mconcas this one also seems to be genuine:

Info in <TUnixSystem::ACLiC>: creating shared library /mnt/mesos/sandbox/sandbox/o2/sw/BUILD/730998ec31b50f6f088b0fb856522e72e0f32253/O2/compiled_macros//mnt/mesos/sandbox/sandbox/o2/sw/SOURCES/O2/6431/0/Detectors/Upgrades/ALICE3/TRK/macros/ALICE3toAO2D_C.so
In file included from input_line_11:6:
/mnt/mesos/sandbox/sandbox/o2/sw/SOURCES/O2/6431/0/Detectors/Upgrades/ALICE3/TRK/macros/ALICE3toAO2D.C:431:13: error: no member named 'addClusterLabelToLayer' in 'o2::its::ROframe'
      event.addClusterLabelToLayer(layer, o2::MCCompLabel(hit.GetTrackID(), iEvent, iEvent, false));
      ~~~~~ ^
Error in <ACLiC>: Dictionary generation failed!

@mconcas this one also seems to be genuine:

Info in <TUnixSystem::ACLiC>: creating shared library /mnt/mesos/sandbox/sandbox/o2/sw/BUILD/730998ec31b50f6f088b0fb856522e72e0f32253/O2/compiled_macros//mnt/mesos/sandbox/sandbox/o2/sw/SOURCES/O2/6431/0/Detectors/Upgrades/ALICE3/TRK/macros/ALICE3toAO2D_C.so
In file included from input_line_11:6:
/mnt/mesos/sandbox/sandbox/o2/sw/SOURCES/O2/6431/0/Detectors/Upgrades/ALICE3/TRK/macros/ALICE3toAO2D.C:431:13: error: no member named 'addClusterLabelToLayer' in 'o2::its::ROframe'
      event.addClusterLabelToLayer(layer, o2::MCCompLabel(hit.GetTrackID(), iEvent, iEvent, false));
      ~~~~~ ^
Error in <ACLiC>: Dictionary generation failed!

Thanks! yes, I was expecting that, will fix that same way (weird that those two did not come up together, would've saved some CI cycles).
Those macros need anyway to be sorted out. @ddobrigk as anticipated, we need to bring upstream something that works. With 3+1D tracking coming soon I guess we'll do a single larger iteration to streamline ALICE 3 tracking.

Btw, @mpuccio will anyway remove macro/run....alice3.C so we will maintain only one in the upgrades dedicated directory

@mconcas : Is this ready to be merged?

@davidrohr : will not change but we are waiting #5911

This PR did not have any update in the last 30 days. Is it still needed? Unless further action in will be closed in 5 days.

...