Set AllEvent cut string to "always true"

When combining AllEvents with other denoms an error was occurring because AllEvents was an empty string. Now it's set to "1" to work in combination with other denoms.

added bug fix label

assigned to @rjhunter

assigned to @thboettc

added RTA label

requested review from @rjhunter

unassigned @rjhunter

@thboettc

Hi @gunther, nice fix! I take it you have tested that this works out OK on your efficiencies? I assume that ROOT does the right thing and converts 1 -> true, but I've never done this explicitly. If ever I wanted an "always pass" cut I do "(1>0)". Can you put my mind at ease and tell me it works please?

/ci-test

added ci-test-triggered label

• [2022-04-30 18:32] Validation started with lhcb-master-mr#4337

assigned to @rmatev

mentioned in issue Moore#426 (closed)

assigned to @ascarabo and unassigned @thboettc

approved this merge request

resolved all threads

assigned to @mfontana and unassigned @rmatev and @ascarabo

@mfontana this is ready for final testing, no hurdles expected

/ci-test

• [2022-05-11 10:40] Validation started with lhcb-master-mr#4385
• [2022-05-31 22:17] Validation started with lhcb-master-mr#4586

mentioned in issue Moore#432 (closed)

assigned to @cagapopo and unassigned @mfontana

assigned to @rmatev

Tests look fine, I'm handing this over to you @rmatev

resolved all threads

assigned to @sponce

added 8 commits

• 1b162440...cb29d128 - 7 commits from branch master
• de2c4bf2 - fix AllEvent denom

Compare with previous version

mentioned in issue Moore#436 (closed)

@sponce can we merge this?

resolved all threads

merged

mentioned in commit 36b76cab

mentioned in issue Moore#444 (closed)