CPAlgorithms: bugfix in ConfigAccumulator

The excludeFrom argument introduced in !68653 (merged) leads to a crash when it is left to its default value of None, as that one is then set to {} which in Python3 is the empty dictionary - and not the empty set, which is the expected type of excludeFrom.

added bugfix label

Restarted after an emergency Jenkins security update (https://www.jenkins.io/security/advisory/2024-01-24/#SECURITY-3314)

Jenkins please retry a build

This merge request affects 1 package:

• PhysicsAnalysis/Algorithms/AnalysisAlgorithmsConfig

This merge request affects 1 file:

• PhysicsAnalysis/Algorithms/AnalysisAlgorithmsConfig/python/ConfigAccumulator.py

Adding @krumnack ,@tadej as watchers

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

CI Result SUCCESS (hash a0b4480a)

	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 4468]

This is a ConfigAccumulator bugfix, so approving for analysis release purposes.

Giovanni (AR)

removed analysis-review-required label

added analysis-review-approved label

MR looks good to me, approving.

Cheers, Yuriy (L1)

removed review-pending-level-1 label

added review-approved label

merged

mentioned in commit f46fcd97