Removing TrigEgammaPidTools.py as its obsolete in Run 3

This is a 3rd attempt to remove obsolete run 2 files (ATR-22836) from Egamma side this MR deletes TrigEgammaPidTools.py and removes its dependency from

Trigger/TrigAnalysis/TrigEgammaAnalysisTools/python/TrigEgammaAnalysisToolsConfig.py
Trigger/TrigAnalysis/TrigEgammaEmulationTool/python/TrigEgammaEmulationEFConfig.py

changed the description

Hi @fwinkl ,

As I see https://acode-browser1.usatlas.bnl.gov/lxr/search?%21v=head&_filestring=&_string=TrigEgammaPidTools only two files actually import TrigEgammaPidTools

Trigger/TrigAnalysis/TrigEgammaAnalysisTools/python/TrigEgammaAnalysisToolsConfig.py
Trigger/TrigAnalysis/TrigEgammaEmulationTool/python/TrigEgammaEmulationEFConfig.py

but run is failing for two tests

-------------------------------------------------------------------------------------------------------------------------------------------------
     | test_trigAna_RDOtoRDOTrig_v1Dev_build.py | Reco_tf: 0, CheckLog: 0, MessageCount: 3, RootComp: 169, CountRefComp: 0, ZeroCounts: 0, exit: 0   |
     | test_trigP1_v1Dev_build.py               | athenaHLT: 0, CheckLog: 10, MessageCount: 2, RootComp: 168, ZeroCounts: 1, exit: 10                |
     | test_trig_data_newJO_build.py            | athena_run: 0, ExecSummary: 0, CheckLog: 0, MessageCount: 0, RootComp: 172, ZeroCounts: 0, exit: 0 |
     | test_trig_data_v1Dev_build.py            | athena: 137, CheckLog: 0, MessageCount: 0, RootComp: 0, CountRefComp: 0, ZeroCounts: 0, exit: 137  |
-------------------------------------------------------------------------------------------------------------------------------------------------

The directory of log file /afs/cern.ch/work/d/dbakshig/public/mvRun2/run/runTrigART_rmRun2 ; I don't see a clear error message

added 1 commit

• fcb54c0d - Modifying TrigEgammaAnalysisToolsConfig and TrigEgammaEmulationEFConfig to rm...

Compare with previous version

marked this merge request as ready

This merge request affects 3 packages:

• Trigger/TrigAnalysis/TrigEgammaAnalysisTools
• Trigger/TrigAnalysis/TrigEgammaEmulationTool
• Trigger/TrigHypothesis/TrigEgammaHypo

Affected files list will not be printed in this case

Adding @dbakshig ,@cjmeyer ,@fernando as watchers

added Egamma Trigger master review-pending-level-1 labels

CI Result SUCCESS (hash fcb54c0d)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional tests

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

resolved all threads

The changes are fine and there are no CI issues. Approving.

Jason (L1)

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

merged

mentioned in commit 4a64a90c

added sweep:ignore label