CPAlgorithms: bugfix for the PerEventSF block

Upon upgrading TopCPToolkit to the ConfigFactory formalism, I noticed that we can't actually use name as an argument of any config block that goes into the factory, since makeConfig itself has a name argument and will load the config blocks arguments as additional kwargs. This MR fixes that.

Since algName is also used in the ConfigFactory, I'm also changing algName to algoName in the config block to avoid any confusion.

changed the description

added bugfix label

This merge request affects 1 package:

• PhysicsAnalysis/Algorithms/AsgAnalysisAlgorithms

This merge request affects 1 file:

• PhysicsAnalysis/Algorithms/AsgAnalysisAlgorithms/python/AsgAnalysisConfig.py

Adding @tadej ,@krumnack as watchers

added Analysis CPAlgorithms analysis-review-required main review-pending-level-1 labels

CI Result SUCCESS (hash eea93924)

	Athena	AnalysisBase	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
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 4677]

looks OK from AR side

Giovanni (AR)

removed analysis-review-required label

added analysis-review-approved label

MR looks fine from L1 perspective, CI shows no problems - approving.

Cheers, Yuriy (L1)

removed review-pending-level-1 label

added review-approved label

merged

mentioned in commit 84578c67