-
Notifications
You must be signed in to change notification settings - Fork 14
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
DelphesRootReader_ee_91gev test failing #129
Comments
The test seems to be pretty flaky. I get a roughly 30% failure rate locally. The main issue is that it's not entirely easy to reproduce, because even with a fixed random seed for delphes I sometimes get it to pass sometimes and to fail the other times. |
We can add a |
I didn't even know that is a thing... Let me check quickly whether I find an obvious issue with the seeding just to rule out any stupid issues. |
Although of course it depends on why that test actually fails? Just flaky comparison, or something else. |
OK. RandomSeed setting doesn't seem to be the problem. The issue with |
That missing ctest hook is probably something @jmcarcell can fix. |
It seems to be very consistent in CI, even c.f. #130 |
2/7 Test #5: DelphesRootReader_ee_91gev .....................***Failed 8.78 sec
See log files.
The text was updated successfully, but these errors were encountered: