Small update for the converter#5002
Conversation
| } | ||
|
|
||
| // collision timestamp in ns wrt to the beginning of collision BC | ||
| tsTimeStamp = tsTimeStamp - std::lround(tsTimeStamp / o2::constants::lhc::LHCBunchSpacingNS) * o2::constants::lhc::LHCBunchSpacingNS; |
There was a problem hiding this comment.
Is this considering if the most probably BC which is filled into GlobalBC is off by 2 BCs ?
There was a problem hiding this comment.
No, I think I should correct this to
tsTimeStamp = globalBC * o2::constants::lhc::LHCBunchSpacingNS - tsTimeStamp
The code currently does not account for a filling scheme, so most probable BC == meanBC
There was a problem hiding this comment.
Did Ruben already put the function in the vertex? To get the most probably BC moved by the filling scheme? I think it should be on his side, not yours. Then you just take that value.
There was a problem hiding this comment.
As far as I know, it is not ready yet. At least I do not see the function in PrimaryVertex.h or in Vertex.h (if it even should be in these classes).
*configurable TFNumber
*collision time wrt to the beginning of collision BC