Skip to content
Snippets Groups Projects

TRT_PAI_Process configuration module renaming

Merged TRT_PAI_Process configuration module renaming
jchapman/athena:TRT_PAI_Process_ConfigRenaming_22.0 into 22.0
Merged John Derek Chapman requested to merge jchapman/athena:TRT_PAI_Process_ConfigRenaming_22.0 into 22.0

Follows the changes in !55567 (merged).

The old-style simulation configuration uses ConfiguredFactory. This uses configuration methods defined in ...Config.py modules which are associated with a component name in various PackageNameConfigDb.py modules.

The new ComponentAccumulator-based configuration tends to use ...Config.py as the naming convention for the configuration modules, but in Simulation we have typically used ...ConfigNew.py to avoid a naming clash. This MR renames all the ConfiguredFactory ...Config.py modules under TRT_PAI_Process to ...ConfigLegacy.py. In the second commit we rename the modules containing the CA-based configuration for TRT_PAI_Process packages from ...ConfigNew.py to ...Config.py so that they are consistent with the rest of the CA-based configuration.

NB Please consider the changes in the two commits separately as the combination makes it look like the changes are more complex than they actually are.

Edited by John Derek Chapman

Merge request reports

Pipeline #4336870 passed

Stage: external

    Pipeline passed for 45a694c0 on jchapman:TRT_PAI_Process_ConfigRenaming_22.0

    Approval is optional

    Merged by Peter BertaPeter Berta 2 years ago (Aug 9, 2022 6:59pm UTC)

    Merge details

    • Changes merged into 22.0 with 514e72be.
    • Deleted the source branch.

    Activity

    Filter activity
    • Approvals
    • Assignees & reviewers
    • Comments (from bots)
    • Comments (from users)
    • Commits & branches
    • Edits
    • Labels
    • Lock status
    • Mentions
    • Merge request status
    • Tracking
    Please register or sign in to reply