TrigInDetValidation: add new ART test for FSLRT chain

Add new ID trigger ART test for FullScan LRT tracking: Trigger/TrigValidation/TrigInDetValidation/test/test_trigID_FSLRT_RHadron.py

other modified files:

modified: Trigger/TrigAnalysis/TrigInDetAnalysisExample/src/AnalysisConfigMT_Ntuple.cxx

modified: Trigger/TrigAnalysis/TrigInDetAnalysisUser/share/TIDAdata-chains-run3.dat

modified: Trigger/TrigValidation/TrigInDetValidation/python/TrigInDetArtSteps.py

modified: Trigger/TrigValidation/TrigInDetValidation/share/TrigInDetValidation_AODtoTrkNtuple.py

modified: Trigger/TrigValidation/TrigInDetValidation/share/TrigInDetValidation_Base.py

modified: Trigger/TrigValidation/TrigInDetValidation/share/comparitor.json

added 1 commit

• 4a96e79c - remove empty lines

Compare with previous version

marked this merge request as ready

This merge request affects 3 packages:

• Trigger/TrigAnalysis/TrigInDetAnalysisExample
• Trigger/TrigAnalysis/TrigInDetAnalysisUser
• Trigger/TrigValidation/TrigInDetValidation

Affected files list will not be printed in this case

Adding @sutt ,@hartj ,@okumura ,@vmartin ,@nagano ,@mvozak as watchers

added Trigger master review-pending-level-1 labels

CI Result SUCCESS (hash 4a96e79c)

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

Setting review-user-action-required until the open threads are resolved. Tom - L1 shifter

added review-user-action-required label and removed review-pending-level-1 label

marked this merge request as draft

added 1 commit

• 7bc2e044 - Use "DTE" in chainname rather than a new "requiredDecision" flag

Compare with previous version

added 1 commit

• bc83c218 - remove debug printout

Compare with previous version

added 1 commit

• e500e166 - remove superfluos "requireDecision" check

Compare with previous version

added 1 commit

• 66191410 - fix args for InDetAna step

Compare with previous version

added 1 commit

• db893de2 - remove white space

Compare with previous version

added 1 commit

• e8596a52 - remove "reqDec" and use "DTE" for FS LRT chains

Compare with previous version

marked this merge request as ready

Hi @sutt , I've made the changes as discussed. Hopefully this now looks OK to you? Cheers Julie

This merge request affects 3 packages:

• Trigger/TrigAnalysis/TrigInDetAnalysisExample
• Trigger/TrigAnalysis/TrigInDetAnalysisUser
• Trigger/TrigValidation/TrigInDetValidation

Affected files list will not be printed in this case

Adding @sutt ,@hartj ,@okumura ,@vmartin ,@nagano ,@mvozak as watchers

added review-pending-level-1 label and removed review-user-action-required label