Read JetDS configuration from chainDict

Avoid parsing the chain name in favour of querying the chainDict. This was never read from the jet chainParts, so remove it and the defunct 'TLA' expression. Reading from the full dict rather than chainParts is preferable in any case, as it is then defined only once per chain, regardless of how many jet legs are present.

As gscThreshold no longer exists, I also took advantage of this opportunity to remove the lines that parse it.

Attn: @valentem, @jbossios, @doglioni

This merge request affects 1 package:

• Trigger/TriggerCommon/TriggerMenuMT

This merge request affects 3 files:

• Trigger/TriggerCommon/TriggerMenuMT/python/HLTMenuConfig/Jet/JetChainConfiguration.py
• Trigger/TriggerCommon/TriggerMenuMT/python/HLTMenuConfig/Menu/DictFromChainName.py
• Trigger/TriggerCommon/TriggerMenuMT/python/HLTMenuConfig/Menu/SignatureDicts.py

Adding @sutt ,@ademaria ,@iriu ,@carquin ,@dzanzi ,@bernius ,@hrussell ,@malconad as watchers

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

CI Result SUCCESS (hash d965521f)

	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 1
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 32839]

added 1 commit

• a2fd650d - Remove unused var

Compare with previous version

This merge request affects 1 package:

• Trigger/TriggerCommon/TriggerMenuMT

This merge request affects 3 files:

• Trigger/TriggerCommon/TriggerMenuMT/python/HLTMenuConfig/Jet/JetChainConfiguration.py
• Trigger/TriggerCommon/TriggerMenuMT/python/HLTMenuConfig/Menu/DictFromChainName.py
• Trigger/TriggerCommon/TriggerMenuMT/python/HLTMenuConfig/Menu/SignatureDicts.py

Adding @sutt ,@ademaria ,@iriu ,@carquin ,@dzanzi ,@bernius ,@hrussell ,@malconad as watchers

CI Result SUCCESS (hash a2fd650d)

	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 32851]

cc @mmontell who is implementing a similar chain for photons in !42984 (merged), following the jet example

And on this MR, let me just say that I like the if self.dict["eventBuildType"]=="JetDS" solution - didn't think of it earlier, but now I'd say it should be recommended. Thanks, TJ!

Changes and CI look good. Approving.

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

mentioned in merge request !42984 (merged)

merged

mentioned in commit 789690bf

added sweep:ignore label