BPHY13 and BPHY23 Updates (derivations only)

Updated the BPHY13.py to enable reading of MuonSegments in the DAOD output files. Updated JpsiPlusPsiCascade., PsiPlusPsiCascade. and BPHY23.py to deal with large number of candidates per event.

added 21.2 Derivation labels

This merge request affects 1 package:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys

This merge request affects 6 files:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys/DerivationFrameworkBPhys/JpsiPlusPsiCascade.h
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys/DerivationFrameworkBPhys/PsiPlusPsiCascade.h
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys/share/BPHY13.py
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys/share/BPHY23.py
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys/src/JpsiPlusPsiCascade.cxx
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys/src/PsiPlusPsiCascade.cxx

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

added review-pending-level-1 label

This merge request affects 1 package:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys

This merge request affects 6 files:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys/DerivationFrameworkBPhys/JpsiPlusPsiCascade.h
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys/DerivationFrameworkBPhys/PsiPlusPsiCascade.h
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys/share/BPHY13.py
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys/share/BPHY23.py
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys/src/JpsiPlusPsiCascade.cxx
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys/src/PsiPlusPsiCascade.cxx

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

The system determined that CI tests (with names matching "^CITest_SimulationRun(2|3)AF3.*$") are not needed for this code change. They are not run. This is not an indicator to restart the job.

Two CI jobs are triggered by 21.2 MR events: for AthDerivation with gcc62 compiler and with gcc8 for AnalysisBase and AthAnalysis projects.

• Developers and shifters need to wait for both results, sometimes for additional hour or two.

CI Result SUCCESS (hash 2e6b5b91)

	AnalysisBase	AthAnalysis
externals
cmake
make
tests

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

The system determined that CI tests (with names matching "^CITest_SimulationRun(2|3)AF3.*$") are not needed for this code change. They are not run. This is not an indicator to restart the job.

CI Result SUCCESS (hash 2e6b5b91)

	AthDerivation
externals
cmake
make
tests

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

CI warnings are unrelated and changes look fine to me, approving
Albert (L1)

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

@boeriu @calpigia: This is only affecting derivations.

mentioned in commit 1c07a812

merged

added sweep:ignore label