Skip to content

Commit

Permalink
k4run: make configurables a frozenset
Browse files Browse the repository at this point in the history
  • Loading branch information
Zehvogel committed Aug 29, 2023
1 parent ef7f9cb commit 7df4783
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion k4FWCore/scripts/k4run
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,9 @@ def load_file(file):
def add_arguments(parser, app_mgr):
# length increases when properties of an algorithm with tools are inspected
# see https://github.com/key4hep/k4FWCore/pull/138
for conf in tuple(app_mgr.allConfigurables.values()):
# can contain the same value multiple times
# see https://github.com/key4hep/k4FWCore/issues/141
for conf in frozenset(app_mgr.allConfigurables.values()):
# skip public tools and the applicationmgr itself
if "ToolSvc" in conf.name() or "ApplicationMgr" in conf.name():
continue
Expand Down

0 comments on commit 7df4783

Please sign in to comment.