-
Notifications
You must be signed in to change notification settings - Fork 628
PWGUD Table Producers modified to include the UPC Reco flag in collisions #7975
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
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
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. |
…ions (AliceO2Group#7975) * Add files via upload * Add files via upload * Add files via upload * Add files via upload * Please consider the following formatting changes * Fix error in UPCCandProducer when building O2Physics I assume the error is due to BCFlags table not to be joined with BC table * Update UDCollisionsConverter.cxx * Update DGBCCandProducer.cxx * Update DGCandProducer.cxx * Update SGCandProducer.cxx * Update DGCandProducer.cxx * Update UPCCandidateProducer.cxx * Update DGBCCandProducer.cxx * Update DGCandProducer.cxx * Update SGCandProducer.cxx * Please consider the following formatting changes * Update UPCCandidateProducer.cxx All BCs should have now have the same table definitions * Update SGCandProducer.cxx * Update DGCandProducer.cxx * Update DGBCCandProducer.cxx * Please consider the following formatting changes * Update UPCCandidateProducer.cxx * Please consider the following formatting changes * Update UPCCandidateProducer.cxx Fix typo probably introduced when global replace was done * Update DGBCCandProducer.cxx * Update DGCandProducer.cxx * Update SGCandProducer.cxx * Update DGBCCandProducer.cxx fix of Vertex::UPCMode * Update DGCandProducer.cxx fix of Vertex::UPCMode * Update SGCandProducer.cxx fix of Vertex::UPCMode * Update UPCCandidateProducer.cxx 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. * Update DGBCCandProducer.cxx fix * Update DGCandProducer.cxx fix * Update SGCandProducer.cxx fix * Update DGBCCandProducer.cxx fix * Update DGCandProducer.cxx fix * Update DGCandProducer.cxx fix * Update DGBCCandProducer.cxx * Update DGBCCandProducer.cxx * Update DGCandProducer.cxx * Update SGCandProducer.cxx --------- Co-authored-by: ALICE Action Bot <alibuild@cern.ch> Co-authored-by: rolavick <roman.lavicka@cern.ch>
No description provided.