-
Notifications
You must be signed in to change notification settings - Fork 483
add best knowldge of collision time in tof matching info #14615
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
REQUEST FOR PRODUCTION RELEASES: This will add The following labels are available |
c9cc223 to
9b88cf3
Compare
08b6f3c to
82da473
Compare
|
@miranov25 , This PR provide FT0 info already coupled to TOF matching info so that you can use it without any dependence on FT0 recPoints (which means it is easier to be used in other task without any extra requirement). |
|
All checks were passed. |
|
Hello Francesco. Before mering, I have 2 question:
|
|
Hi @noferini Thanks, this would be very useful! But I think for PbPb one should 1st apply FT0 clean up, to get rid of the QED triggers. You can see the spectrum or FT0A,C amplitudes and number of triggers passing your selection for the sample were only 138661 primary vertices were found: You can use the same selection I apply in the ITS/TPC matching to contain the TPC tracks time:
Note that the |
|
Hi @shahor02 , Are you suggestion this for a computational time or to reduce contamination. For contamination issue: I would like to stress that FT0 is used here only if
Under these conditions I would not expect issue. |
|
@noferini my concern is the contamination by QED triggers. If you want, we can merge it as is, the check on the real data updated TOFInfo from your code with the one hacked by the selection above. |
|
Hi @shahor02 , |
shahor02
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@noferini looks fine to me, shall I merge it?
|
Hi @shahor02 , yes we can merge. Thanks @miranov25 : concerning the small shift it should be investigated. Only recently we managed to fix an issue with FT0 in MC to have the collision time in the right BC but I have the feeling that there are second order effects which has to be tuned (I will check on our side after the Pb-Pb run but I suspect it can be on FT0 side) |
|
Thank you. |




Dear @shahor02 , @miranov25 ,
I am preparing a PR to attach the best FT0 info already in the TOF matching info since the info is available in the final stage of TOF matching (it is needed for TOF calibInfo).
I added one double (collision time) and one float (T0 res.) in the TOF matching info (whose output is in any case not persistent)
Once ready it should be easy for you to use it.