DerivationBPHY: Add warnings for excess candidates

Sometimes combinatoric explosions crash the shared writer. Adding warnings for easier debugging.

cc @wwalko @kfung

added 1 commit

• 22604f78 - Add CandidateLimit to prevent explosive combinatorics

Compare with previous version

added 1 commit

• 88e0c7c6 - Add CandidateLimit to prevent explosive combinatorics

Compare with previous version

added 1 commit

• a27ff546 - Add CandidateLimit to prevent explosive combinatorics

Compare with previous version

marked this merge request as ready

Jenkins please retry a build

This merge request affects 2 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys
• PhysicsAnalysis/JpsiUpsilonTools

Affected files list will not be printed in this case

Adding @wwalko ,@emmat ,@abarton ,@calpigia ,@jcatmore as watchers

added Analysis Derivation main review-pending-level-1 labels

CI Result SUCCESS (hash a27ff546)

	Athena
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 1180]

Setting review-user-action-required until threads resolved. Tom - L1 shifter

added review-user-action-required label

removed review-pending-level-1 label

added 1 commit

• b0b86982 - Add CandidateLimit to prevent explosive combinatorics

Compare with previous version

This merge request affects 2 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys
• PhysicsAnalysis/JpsiUpsilonTools

Affected files list will not be printed in this case

Adding @abarton ,@wwalko ,@emmat ,@calpigia ,@jcatmore as watchers

added review-pending-level-1 label and removed review-user-action-required label

CI Result SUCCESS (hash b0b86982)

	Athena
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 1229]

There are no issues with the CI but there are unresolved thread. Switching tags until these are resolved.

-- James for L1

added review-user-action-required label and removed review-pending-level-1 label

resolved all threads

removed review-user-action-required label

added review-pending-level-1 label

Changes are simple and passes CI. Approving for L1.

added review-approved label and removed review-pending-level-1 label

mentioned in commit 28234626

merged