Repository navigation
[PWGCF] Add internal validation and new cuts - #18522
Conversation
|
O2 linter results: ❌ 0 errors, |
|
Error while checking build/O2Physics/code-check for ccadc27 at 2026-10-30 12:24: Full log here. |
| } misc; | ||
|
|
||
| struct InternalValidation { | ||
| TRandom3* randGen = new TRandom3(1); |
There was a problem hiding this comment.
Why do you have here new TRandom3(1); and not new TRandom3(0);? Only if you use the latter, ROOT ensures that the seed for random generation is unique in time and space.
There was a problem hiding this comment.
Because initially I want to ensure that the seed is the same for each time I run it, just like in MC studies. But I didn't think too much for hyperloop case...
I have changed it to 0. Thank you very much for the correction!
There was a problem hiding this comment.
Not now, but in the next PR you could add a configurable option for this argument to cover both use cases.
abilandz
left a comment
There was a problem hiding this comment.
Just please replace quickly new TRandom3(1); with new TRandom3(0)- that's relvant if you want to redo this validation study in Hyperloop.
Add internal validation for verifying Q-vector calculations.
Add additional experimental particle cuts.
Add numContrib hist.
Simplification of naming of some variable.