Remove step number from getEmptyStep method
This MR removes the step number argument from the ChainConfigurationBase.getEmptyStep
method (ATR-30694). This number was used to add a Step#_
prefix to the empty steps' name. However, any pre-existing prefix is overridden during the chain merging and menu alignment process, so there is no need for the manual prefix.
I compared the HLTMenu and HLTJobOptions json files, and there are no differences after this change.
Merge request reports
Activity
This merge request affects 1 package:
- Trigger/TriggerCommon/TriggerMenuMT
This merge request affects 9 files:
- Trigger/TriggerCommon/TriggerMenuMT/python/HLT/CalibCosmicMon/CalibChainConfiguration.py
- Trigger/TriggerCommon/TriggerMenuMT/python/HLT/Config/ChainConfigurationBase.py
- Trigger/TriggerCommon/TriggerMenuMT/python/HLT/Electron/ElectronChainConfiguration.py
- Trigger/TriggerCommon/TriggerMenuMT/python/HLT/Jet/JetChainConfiguration.py
- Trigger/TriggerCommon/TriggerMenuMT/python/HLT/MinBias/MinBiasChainConfiguration.py
- Trigger/TriggerCommon/TriggerMenuMT/python/HLT/Muon/MuonChainConfiguration.py
- Trigger/TriggerCommon/TriggerMenuMT/python/HLT/Tau/TauChainConfiguration.py
- Trigger/TriggerCommon/TriggerMenuMT/python/HLT/Test/TestChainConfiguration.py
- Trigger/TriggerCommon/TriggerMenuMT/python/HLT/UnconventionalTracking/UnconventionalTrackingChainConfiguration.py
Adding @sutt ,@martindl ,@ademaria ,@miochoa ,@bkerridg ,@cantel ,@jbeaucam ,@iriu ,@shanisch ,@lidiaz ,@gipezzul ,@carquin ,@slai as watchers
added 24.0 Egamma JetEtmiss Tau Trigger TriggerMenu TriggerMinBias review-pending-level-1 labels
CI Result SUCCESS (hash f59a3375)Athena 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
For experts only: Jenkins output (remote access info)added review-approved label and removed review-pending-level-1 label
added review-approved-tier0 label
Thanks - looks like a needed cleanup. cc @fpastore.
added review-approved-point1 label
mentioned in commit 3b371405
mentioned in merge request !77151 (merged)