Skip to content

CPAlgorithms: bugfix for the PerEventSF block

Baptiste Ravina requested to merge ravinab/athena:ravinab-main-patch-f6b5 into main

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.

Edited by Baptiste Ravina

Merge request reports