AnalysisTop: Fixing an issue when if an empty map is provided the error printed is confusing

Fixing a bug in the code where tight triggers were passed for loose setting and an empty map was identified as a photon trigger.

This merge request affects 1 package:

• PhysicsAnalysis/TopPhys/xAOD/TopCPTools

This merge request affects 1 file:

• PhysicsAnalysis/TopPhys/xAOD/TopCPTools/Root/TopTriggerCPTools.cxx

Adding @bmondal ,@spalazzo ,@mvanadia ,@omajersk ,@tdado as watchers

added 21.2 Analysis AnalysisTop Trigger review-pending-level-1 labels

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 612a168b)

	AnalysisBase	AthAnalysis
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 1
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 31098]

CI Result SUCCESS (hash 612a168b)

	AthDerivation
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
AthDerivation: number of compilation errors 0, warnings 17
For experts only: Jenkins output [CI-MERGE-REQUEST-GCC62-CC7 656]

Build warnings are unrelated. Approving -- L1

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

added 1 commit

• 79c8e14e - Fixing loose configuration

Compare with previous version

changed the description

removed review-approved label

added bugfix review-pending-level-1 labels

This merge request affects 1 package:

• PhysicsAnalysis/TopPhys/xAOD/TopCPTools

This merge request affects 1 file:

• PhysicsAnalysis/TopPhys/xAOD/TopCPTools/Root/TopTriggerCPTools.cxx

Adding @bmondal ,@spalazzo ,@mvanadia ,@omajersk ,@tdado as watchers

Jenkins please retry a build

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 79c8e14e)

	AnalysisBase	AthAnalysis
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 6
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 31116]

CI Result SUCCESS (hash 79c8e14e)

	AthDerivation
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
AthDerivation: number of compilation errors 0, warnings 17
For experts only: Jenkins output [CI-MERGE-REQUEST-GCC62-CC7 657]

Hi,

the test finished, please check if you are saitisfied with the change.

Best, Rui AR shifter

Build warnings are unrelated, re-approving, -- L1

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

Hi，

@krumnack Nils, would you please merge this?

Best, Rui AR shifter