ATLASRECTS-7788: Allow deserialisation to skip missing modules when handling multiple HLT results
When processing physics_Main
, we attempt to deserialise all available HLT results, but this means we should not require e.g. the TLA HLT fragment to be present on every event. This is controlled by an option in TrigDeserialiser
that is activated for the algs handling the data scouting fragments.
In the process of fixing this, also adjusting the ExtraInputs/Outputs
to correctly schedule the GapFiller
as suggested by @tamartin. Might help with ATR-28389.
Merge request reports
Activity
added Trigger full-unit-tests main review-pending-level-1 labels
CI Result FAILURE (hash 117b1cea)Athena AthAnalysis externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 341]- Resolved by Jan-Eric Nitschke
I only have one small comment. Otherwise this looks good.
Test failure seems to be unrelated: https://its.cern.ch/jira/browse/ATLASG-2623
Cheers L1
added review-user-action-required label and removed review-pending-level-1 label
- Resolved by Teng Jian Khoo
added 1 commit
- aafcd584 - Copy list arguments to prevent modification of defaults
added review-pending-level-1 label and removed review-user-action-required label
CI Result FAILURE (hash aafcd584)Athena AthAnalysis externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 363]added review-approved label and removed review-pending-level-1 label
Looks good to me now. (Test failures still unrelated https://its.cern.ch/jira/browse/ATLASG-2623)
Cheers L1
mentioned in commit 71653991
added sweep:ignore label