[PWGLF] Jet reconstruction with V0s + MC models process function + Event loss, splitting#16284
Conversation
lorber98
commented
May 18, 2026
- Add histograms to compute event loss and event splitting;
- Add process function to work with MC models (identical to processMCGenerated but without reconstructed collisions);
- Add possibility to include V0s in the input list for the jet finder algorithm. This can be activated with the configurable useV0inJetRec.
|
O2 linter results: ❌ 6 errors, |
| double GetEnergy(double px, double py, double pz, double mass) | ||
| { | ||
| return std::sqrt(px * px + py * py + pz * pz + mass * mass); | ||
| } |
There was a problem hiding this comment.
Why don't you use a RecoDecay method?
There was a problem hiding this comment.
Thank you Vit for the comment, very useful! I'll definitely keep this in mind for future PRs.
Currently, I'm on a very tight schedule to get the next approvals for the analysis. Since this change isn't strictly blocking for the time being, I will implement it as soon as possible in a later PR. I apologize for that and thank you very much for your understanding.
There was a problem hiding this comment.
Fair enough but don't mark comments as resolved, especially if they are not actually resolved.
| * @return int The index of the common mother, or -1 if no common mother exists. | ||
| */ | ||
| template <typename T> | ||
| int GetCommonMotherId(aod::McParticles const& mcParticles, |
There was a problem hiding this comment.
Can you not use RecoDecay::getMother?
There was a problem hiding this comment.
Thank you Vit for the comment, very useful! I'll definitely keep this in mind for future PRs.
Currently, I'm on a very tight schedule to get the next approvals for the analysis. Since this change isn't strictly blocking for the time being, I will implement it as soon as possible in a later PR. I apologize for that and thank you very much for your understanding.
|
Error while checking build/O2Physics/staging for 13e4a4b at 2026-05-18 12:42: Full log here. |
|
Error while checking build/O2Physics/o2 for 13e4a4b at 2026-05-18 12:44: Full log here. |
| if (!isV0) | ||
| continue; | ||
| for (int i = 0; i < fjTracks.size(); ++i) { | ||
| for (int i = 0; i < int(fjTracks.size()); ++i) { |
There was a problem hiding this comment.
Don't do this. You should use an unsigned counter instead.