Changed TRandom to TRandom3 in simu.
All threads resolved!
All threads resolved!
Changed TRandom to TRandom3 in simu. TRandom gives errors sometimes with ROOT version 6.08 when applying smearing to MM hits. This makes some values for the MMhits appear to be nan and as such yield incorrect results for everything based on MMhits reconstructed from the true hit position in MC.
Merge request reports
Activity
requested review from @akorneev
assigned to @bbantoob
- Resolved by Anton Karneyeu
Hi @bbantoob , the gRandom is TRandom3 by default since ages, you can verify with
gRandom->Print()
. So, the change itself looks like no-op. Are you sure it really solve the smearing issue?
Hi @bbantoob , did you managed to reproduce the error with root 6.08?
Edited by Anton Karneyeu
Please register or sign in to reply