Simple macro for ITS tracks vs BC in ROF#15536
Conversation
| int bc = (bcGlobal + nBCsPerOrbit) % nBCsPerITSROF; | ||
| bc -= offsetITSROF; |
There was a problem hiding this comment.
Should not this be (bcGlobal - offsetITSROF + nBCsPerOrbit)%nBCsPerITSROF ?
There was a problem hiding this comment.
The choice of shifting the bc by offsetITSROF after computing the modulus was essentially for visualization purposes: it shifts the dip in the at bc = 0 instead of having it at bc = 64:

If instead I use your calculation of bc = (bcGlobal - offsetITSROF + nBCsPerOrbit)%nBCsPerITSROF, the "dip" is splitted in two "legs": one at bc = 0 and the other one at bc = 198:

A third option could be not to shift by offsetITSROF and have the peak at bc = 64:

The first option seemed to me the one providing the most convenient visualization, but we can switch to the 3rd one if you prefer.
There was a problem hiding this comment.
OK, I was referring to the ITS BC corresponding the LHC BC. I agree that for the visualisation purposes, the dip at 0 is more convenient.
There was a problem hiding this comment.
Do we have another shift that we need to apply to match the ITS BC and the LHC BC, in addition to the 64 BCs shift between the ITS ROF and the other ALICE detectors?
This macro is intended as a starting point example for students doing service work on the ITS tracking vs ROF and the simulation of the time response of the alpide