Jet hypo factorisation to simplify code for FastReduction, removal of obsolete code 2

This MR replaces !MR38552

The code in !MR38552 damages some archaic code still running in various tests, but which had been removed as part of a clean-up.

This MR contains the changes in !MR38552 minus the removing of a number of files.

@jbossios @valentem @khoo @peter

ATR-22411

This merge request affects 1 package:

• Trigger/TrigHypothesis/TrigHLTJetHypo

This merge request affects 7 files:

• Trigger/TrigHypothesis/TrigHLTJetHypo/python/ConditionsToolSetterFastReduction.py
• Trigger/TrigHypothesis/TrigHLTJetHypo/python/ConditionsToolSetterHT.py
• Trigger/TrigHypothesis/TrigHLTJetHypo/python/NodeSplitterVisitor.py
• Trigger/TrigHypothesis/TrigHLTJetHypo/python/TrigJetHypoToolConfig.py
• Trigger/TrigHypothesis/TrigHLTJetHypo/python/chainDict2jetLabel.py
• Trigger/TrigHypothesis/TrigHLTJetHypo/python/testChainDictMaker.py
• Trigger/TrigHypothesis/TrigHLTJetHypo/python/treeVisitors.py

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

Hi @peter , I think the ATR jira ticket in the description is not the one you wanted.

changed the description

ATR ref fixed....

CI Result SUCCESS (hash b66d9453)

	Athena	AthSimulation	AthGeneration	AnalysisBase
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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 24021]

Looks ok, but passing to L2 for further review as changes are fairly extensive.

Cheers Nathan (L1)

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

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

mentioned in commit 169e4ec4

merged

added sweep:ignore label