Moved the lines of b-tagging code that replace or append strings to the top level function BTagAlgsCfg

There are some lines in various packages that replace Track by PV0Track and append Jets to jet collection name. This MR tries to move them to the top level function BTagAlgsCfg. This way the replacement is only done in the top level function.

added BTagging alsoTargeting:22.0 labels

assigned to @rjoshi

marked this merge request as draft

Jenkins please retry a build

This merge request affects 7 packages:

• Control/AthenaConfiguration
• PhysicsAnalysis/AnalysisCommon/ParticleJetTools
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkConfiguration
• PhysicsAnalysis/DerivationFramework/DerivationFrameworkFlavourTag
• PhysicsAnalysis/JetTagging/JetTagAlgs/BTagging
• PhysicsAnalysis/JetTagging/JetTagConfig
• Trigger/TriggerCommon/TriggerMenuMT

Affected files list will not be printed in this case

Adding @sutt ,@nishu ,@rcreager ,@duperrin ,@sanmay ,@jveatch ,@bdong ,@bernius ,@suyogs ,@dguest ,@sekula ,@ssnyder ,@lidiaz as watchers

added Analysis Core Derivation JetEtmiss Trigger TriggerMenu analysis-review-required master labels

There were multiple CI triggers for this MR and commit. The system ignored duplicates but the GitLab pipeline status may incorrectly show the job as failed. Once the remaining job finished running, the CI results will be posted as usual.

CI Result SUCCESS (hash 097f8c68)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional 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
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 56817]

marked this merge request as ready

added analysis-review-approved label and removed analysis-review-required label

Hi. Is there anything that I need to do from my side? Thanks

It isn't clear to me why ATLAS robot didn't add review-pending-level-1. In any case, the changes look fine to me and there are no CI issues. Approving.

Jason (L1)

added review-approved label

merged

mentioned in commit 4a276a6b

added sweep:done label

added sweep:failed label

Sweep summary
failed:

• 22.0

@rjoshi this definitely needs to get swept to 22.0

mentioned in merge request !55726 (merged)