Francesco Noferini

Hi @shahor02 , I was wondering if you can run again on your Pb-Pb apass3 outputs with this PR, and produce o2matchable_tof.root. It would be very useful to start to...

> @noferini shall I merge it 1st or you are still modifying it? @shahor02 , I would not merge it for the moment since it is still wip (I removed...

> @noferini https://cernbox.cern.ch/s/5U24GEsOzVPxltR is the o2matchable_tof.root for 38kHz run LHC23zzk_544508, let me know if you need other data. Many thanks! Running on it

I think we can start the review. Note that in BestMatches I added the possibility (for constrained tracks) to match up to 2 clusters in different strips if times are...

All done from my side. One check (macos) failing due to a git error not related to this PR. Cheers, Francesco

> Thanks @noferini , do you have some plot showing the fakes contamination with new and old method? In principle, the performance are the one reported in my presentation https://indico.cern.ch/event/1406660/contributions/5912002/attachments/2851047/4985149/TOF%20Sim&Reco&QC%20-%207may24.pdf...

Otherwise, I can ask Catalin to run a debug production on the GRID

In the meanwhile I realized that I was storing residuals as used in the new Chi2 definition. I changed it to store original residuals since they may be needed for...

Hi @shahor02 , I prefer to use the full output to be sure that I use exactly the same code. I've already asked @chiarazampolli and @catalinristea if we can do...

At this point I will wait till Monday if I manage to have a full validation on debug production