Configure muon trigger to run on MC
Since for MC we start from RDO input, we don't need to run the BS to RDO decoding to run. Also swtiching off the checking for ID related BS containers if we aren't reading BS input. Fixes the problems reported in ATR-19763. Also add a ART test to run the muon trigger menu on MC.
Merge request reports
Activity
added Trigger TriggerMenu master review-pending-level-1 labels
Hi Savanna,
Would you be able to add an ART test that runs the muon slice on MC? You should be able to just add
doMuonSlice=True
toEXTRA
in here: https://gitlab.cern.ch/atlas/athena/blob/master/Trigger/TrigValidation/TrigUpgradeTest/test/test_trigUpgr_empty_menu_mc_build.shI think eventually we want to move all single-slice tests to MC for statistics, but that's for another time.
Thanks, Stewart
CI Result SUCCESSAthena AthSimulation externals cmake make required tests optional tests Full details available at NICOS MR-23493-2019-05-16-20-06
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST 38569]Hi Stewart,
I can add the test, but was also trying to add the reference comparison since it's probably useful to have and had a question (maybe also for @rbielski and @hartj). Since this is a new test, the reference folder would not be available on cvmfs or eos. Is it better to add the test without the reference comparison first, or add the test with the reference comparison (with the caveat that the comparison will fail at first).
Cheers,
Savanna
CI Result SUCCESSAthena AthSimulation externals cmake make required tests optional tests Full details available at NICOS MR-23493-2019-05-17-01-04
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST 38589]added review-approved label and removed review-pending-level-1 label
mentioned in commit 81b2f5dd
added sweep:ignore label