Update testRandomSpacing

This change should fix the current issues with testRandomSpacing, I Replaced the std to maximum between expected and current number of counts to prevent failing while it is actually statistically fine.

This should fix the testRandomSpacing issue in #607 (closed).

I am not sure I understand the change. Could you describe it a bit more?

It has to do with the fact that we are calculating the deviation from the expected number of events and not from the number of events we got. If we do the maximum between both than we prevent cases in which we have drawn 6 numbers but the expectation is a bit less than 1, in that case the std is given by 5, that results in 6 > .99 + 5, than the code returns an error because it is not within 5 sigma of the expectation but the std of 6 is around 2.5 so that is constistent with an expectation of 1 within 3 sigma. I have seen this error also for other number of particles like 7, 8 and 9. For lower numbers it is not possible because I required the deviation to be at least 5 in all cases.

I see. Can you then put in a comment in the code explaining the logic behind this?

added 1 commit

• 7f199536 - Clarify testRandomSpacing by adding a few more comments

Compare with previous version

I added a few extra comments in the code to clarify it.

Thanks! Very clear comment.

merged

mentioned in commit 0902c43c