Skip to content

Commit

Permalink
Merge pull request #256 from ecmwf-ifs/naml-scheduler-fix-dependency-…
Browse files Browse the repository at this point in the history
…trafo

DependencyTransformation: Ensure tuples when adding "block" entries
  • Loading branch information
reuterbal authored Mar 21, 2024
2 parents 9e9066f + 07a7883 commit 8bb2217
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions loki/transform/dependency_transform.py
Original file line number Diff line number Diff line change
Expand Up @@ -254,7 +254,7 @@ def _update_item(orig_name, new_name):
if item and (matched_keys := SchedulerConfig.match_item_keys(orig_name, item.ignore)):
# Add the renamed but ignored items to the block list because we won't be able to
# find them as dependencies under their new name anymore
item.config['block'] = item.block + tuple(
item.config['block'] = as_tuple(item.block) + tuple(
new_name for name in item.ignore if name in matched_keys
)
item.config['ignore'] = tuple(
Expand Down Expand Up @@ -475,7 +475,7 @@ def _update_item(proc_name, module_name):
if item and (matched_keys := SchedulerConfig.match_item_keys(proc_name, item.ignore)):
# Add the module wrapped but ignored items to the block list because we won't be able to
# find them as dependencies under their new name anymore
item.config['block'] = item.block + tuple(
item.config['block'] = as_tuple(item.block) + tuple(
module_name for name in item.ignore if name in matched_keys
)

Expand Down

0 comments on commit 8bb2217

Please sign in to comment.