add HLT emulation test using menu code (emu_step_menu test)
We are adding one additional ART test that configures a test menu with TestChains. The latter use emulation algorithms and emulated data, similarly to EmuStepProcessing test. They have common dummy input data and try to reproduce the same processing chains. Main changes are:
- test_trigUpgr_emu_step_menu_processing_build.sh: script to launch the new test
- EmuStepProcessingTest.py: JO file of the test: now can be configured with “doMenu" flag (false by default), so that both tests can be launched with the same JO py file
- EmuStepProcessingConfig.py: contains configurations for both tests. The new test is overwriting the default menu LS2_v1 with the new one, LS2_emu_v1, before building the chains
- LS2_emu_v1.py: menu file for the new emulation test (TestSlice used only)
- TestDef.py: definition of Test signature code (following the same approach as all other signatures)
This is related to ATR-21589 tagging @tbold, @tamartin , @aranzazu , @ckaldero, @markowen
This should also fix ATR-21378
Merge request reports
Activity
added Trigger TriggerMenu master review-pending-level-1 labels
removed Trigger TriggerMenu master review-pending-level-1 labels
Also tagging @fwinkl as he is optimising tests in TMMT.
- Resolved by Frank Winklmeier
Since we want to abandon TrigUpgradeTest, is it really a good idea to add new tests there? cc @rbielski
CI Result SUCCESS (hash c05914bb)Athena AthSimulation AnalysisBase AthGeneration 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
AnalysisBase: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 15640]Somehow this MR has no labels, so it's not seen by shifters. Not sure if the removal was intentional or due to a bug in GitLab. I'm restoring them now.
Edited by Rafal Bielskiadded Trigger TriggerMenu master review-pending-level-1 labels
added review-pending-level-2 label and removed review-pending-level-1 label
- Resolved by Edward Moyse
I've had a look at this, and it seems fine to me. Also, it seems like Trigger experts have also checked. So I'll accept this shortly, if no one objects.
Edited by Edward Moyse
added review-approved label
removed review-pending-level-2 label
mentioned in commit ba578ab8
added sweep:ignore label