TrigInDetValidation: remove LRT specific flags and add more general ones

Remove LRT specific flags

For RDO->AOD step add preclude file. Preinclude file in the test itself now need to include step (previously we assumed it was RDOtoRDOTrigger)

Instead of LRT flag for the analysis step add a more general "ExtraAna" input.

Change some test names so everything is now lowercase

marked this merge request as ready

added 1 commit

• 5c14545b - change if

Compare with previous version

added 1 commit

• 05c933be - Change names to lowercase for some tests

Compare with previous version

Hi @sutt, I didn't realise you'd already made some of these changes in !42002 (merged)

My changes use "ExtraAna" rather than "Extra" for the extra args (I thought this indicated that these are for the analysis step rather than some other step) Want to check tat this is OK for you before I resolve the conflicts

Thanks Julie

changed the description

I guess so, although for my information, why have the LRT tests been deleted ? Are they no longer needed ? Or have they been superceded by something else ?

Mark

Hi @sutt Oopops didn't add the tests with lowercase names. Should be there now. Thanks for spotting that one!

Cheers Julie

added 1 commit

• 5c7fe3d0 - actually add the tests with lowercase names

Compare with previous version

added 297 commits

• 5c7fe3d0...11f37e2f - 296 commits from branch atlas:master
• ddb8c03f - Merge remote-tracking branch 'upstream/master' into master-TIDV-art66

Compare with previous version

This merge request affects 1 package:

• Trigger/TrigValidation/TrigInDetValidation

This merge request affects 16 files:

• Trigger/TrigValidation/TrigInDetValidation/python/TrigInDetArtSteps.py
• Trigger/TrigValidation/TrigInDetValidation/share/TIDAlrt_preinclude.py
• Trigger/TrigValidation/TrigInDetValidation/share/TrigInDetValidation_Base.py
• Trigger/TrigValidation/TrigInDetValidation/test/test_trigID_FSLRT_RHadron.py
• Trigger/TrigValidation/TrigInDetValidation/test/test_trigID_bjet_nopps_pu40.py
• Trigger/TrigValidation/TrigInDetValidation/test/test_trigID_el_zee_clones_pu40.py
• Trigger/TrigValidation/TrigInDetValidation/test/test_trigID_fsjet_ml2_pu40.py
• Trigger/TrigValidation/TrigInDetValidation/test/test_trigID_fsjet_ml_pu40.py
• Trigger/TrigValidation/TrigInDetValidation/test/test_trigID_fsjet_nopps_pu40.py
• Trigger/TrigValidation/TrigInDetValidation/test/test_trigID_fsjet_vtx_pu40.py
• Trigger/TrigValidation/TrigInDetValidation/test/test_trigID_fslrt_rhadron.py
• Trigger/TrigValidation/TrigInDetValidation/test/test_trigID_minbias.py
• Trigger/TrigValidation/TrigInDetValidation/test/test_trigID_muLRT_StauStau.py
• Trigger/TrigValidation/TrigInDetValidation/test/test_trigID_muLRT_Zmumu_pu40.py
• Trigger/TrigValidation/TrigInDetValidation/test/test_trigID_mulrt_staustau.py
• Trigger/TrigValidation/TrigInDetValidation/test/test_trigID_mulrt_zmumu_pu40.py

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

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

resolved all threads

resolved all threads

CI Result SUCCESS (hash ddb8c03f)

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

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

mentioned in commit d03707b6

merged

added sweep:ignore label