-
Notifications
You must be signed in to change notification settings - Fork 497
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
PWGUD Table Producers modified to include the UPC Reco flag in collisions #7975
base: master
Are you sure you want to change the base?
Conversation
Please consider the following formatting changes to AliceO2Group#7975
Error while checking build/O2Physics/o2 for 351608d at 2024-10-12 23:50:
Full log here. |
I assume the error is due to BCFlags table not to be joined with BC table
Error while checking build/O2Physics/o2 for e92b048 at 2024-10-13 18:22:
Full log here. |
Error while checking build/O2Physics/o2 for 33e487b at 2024-10-13 20:28:
Full log here. |
Please consider the following formatting changes to AliceO2Group#7975
Error while checking build/O2Physics/o2 for f681483 at 2024-10-14 10:56:
Full log here. |
All BCs should have now have the same table definitions
Please consider the following formatting changes to AliceO2Group#7975
Please consider the following formatting changes to AliceO2Group#7975
Fix typo probably introduced when global replace was done
Error while checking build/O2Physics/o2 for b831ae0 at 2024-10-14 14:16:
Full log here. |
fix of Vertex::UPCMode
fix of Vertex::UPCMode
fix of Vertex::UPCMode
Error while checking build/O2Physics/o2 for f9417a4 at 2024-10-14 16:02:
Full log here. |
TODO added to not forget the implementation of collision flag association via bc table is not fully correct. bcs.iteratorAt(0) takes only first BC in a timeframe, and in an analysis task data should be processed TF-by-TF (if I'm not mistaken again), so it should be a corresponding BCId for each collision. UPCCandidateProducer matches tracks by time (BC) independently of reconstructed vertices, so additional checks should be added at some point to determine settings for a given ITSROF based e.g. on track BCs.
Error while checking build/O2Physics/o2 for a1da96e at 2024-10-14 19:08:
Full log here. |
No description provided.