Repository navigation
higher MB gaps for testing - #2515
jaimenorman wants to merge 2 commits into
Conversation
|
REQUEST FOR PRODUCTION RELEASES: This will add The following labels are available |
|
Hi @jaimenorman, thanks a lot for the quick PR right after JSUB discussion. The simulation test fails because of the test size. With the default 100 events, gap 9 gives 12 jet-jet events (events 0, 9, ..., 99) and gap 11 gives 10, outside the ±5 % window of the test macros (10.6–11.7 and 8.6–9.5). Gap 7 gives 15 and passes at the edge (13.6–15.0). With a multiple of the gap as the number of test events (98, 99, 99), all three give exactly 1/gap. |
|
Thanks a lot @joonsukbae for spotting this, I have increased the tolerance in the check. |
No description provided.