Skip to content

Remove “1” and “N” designations in hypo name (ATR-22834)

Daniele Zanzi requested to merge dzanzi/athena:ATR-22834_dzanzi into master

This should address ATR-22834, but since this is code I am not familiar with I would ask @hrussell @khoo @tamartin to check. CI tests are successful and now I am checking that the counts are not changed. However, if this breaks some combined chains we may not see count changes in the local tests...

With this fix, the steps

        "Step1_1FastCalo_electron": [
            "RoRSeqFilter/FStep1_1FastCalo_electron",
            "EventViewCreatorAlgorithm/IMfastCalo",
            "AthViews::ViewDataVerifier/fastCaloVDV",
            "T2CaloEgammaReFastAlgo/FastCaloL2EgammaAlg",
            "TrigEgammaFastCaloHypoAlg/ElectronEgammaFastCaloHypo",
            "ComboHypo/ComboHypo_Step1_1FastCalo_electron"
        ],
        "Step1_NFastCalo_electron": [
            "RoRSeqFilter/FStep1_NFastCalo_electron",
            "EventViewCreatorAlgorithm/IMfastCalo",
            "AthViews::ViewDataVerifier/fastCaloVDV",
            "T2CaloEgammaReFastAlgo/FastCaloL2EgammaAlg",
            "TrigEgammaFastCaloHypoAlg/ElectronEgammaFastCaloHypo",
            "ComboHypo/ComboHypo_Step1_NFastCalo_electron"
        ],

are now the same

        "Step1_FastCalo_electron": [
            "RoRSeqFilter/FStep1_FastCalo_electron",
            "EventViewCreatorAlgorithm/IMfastCalo",
            "AthViews::ViewDataVerifier/fastCaloVDV",
            "T2CaloEgammaReFastAlgo/FastCaloL2EgammaAlg",
            "TrigEgammaFastCaloHypoAlg/ElectronEgammaFastCaloHypo",
            "ComboHypo/ComboHypo_Step1_FastCalo_electron"
        ],
Edited by Daniele Zanzi

Merge request reports