Skip to content

Commit

Permalink
Added logging, fixed name
Browse files Browse the repository at this point in the history
  • Loading branch information
leventeBajczi committed Nov 25, 2024
1 parent a707842 commit 52a207e
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion contrib/slurm/arrayexecutor.py
Original file line number Diff line number Diff line change
Expand Up @@ -231,7 +231,8 @@ def filter_previous_results(run_set, benchmark):
missing_runs = []
for run in run_set.runs:
props = " ".join(sorted([prop.name for prop in run.properties]))
key = (run.name, props)
name = relative_path(run.identifier, result_file)
key = (name, props)
if key in previous_runs:
old_log = str(
os.path.join(logfile_folder, os.path.basename(run.identifier) + ".log")
Expand All @@ -249,10 +250,13 @@ def filter_previous_results(run_set, benchmark):

run.set_result(previous_runs[key])
else:
logging.warning(f"Old files directory {old_files} does not exist. Skipping run {name}.")
missing_runs.append(run)
else:
logging.warning(f"Old log {old_log} does not exist. Skipping run {name}.")
missing_runs.append(run)
else:
logging.warning(f"Run with key {key} not found in results. Skipping run {name}.")
missing_runs.append(run)

shutil.rmtree(logfile_folder)
Expand Down

0 comments on commit 52a207e

Please sign in to comment.