Skip to content

[scoreboard] Correct efficiency/acceleration for seq version #550

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
allnes merged 1 commit into from
Jul 21, 2025
Merged

[scoreboard] Correct efficiency/acceleration for seq version #550

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 10 additions & 4 deletions scoreboard/main.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ def load_performance_data(perf_stat_file_path):
return perf_stats


def calculate_performance_metrics(perf_val, eff_num_proc):
def calculate_performance_metrics(perf_val, eff_num_proc, task_type):
"""Calculate acceleration and efficiency from performance value."""
acceleration = "?"
efficiency = "?"
Expand All @@ -73,8 +73,14 @@ def calculate_performance_metrics(perf_val, eff_num_proc):
perf_float == float("inf") or perf_float != perf_float
):
speedup = 1.0 / perf_float
acceleration = f"{speedup:.2f}"
efficiency = f"{speedup / eff_num_proc * 100:.2f}%"
# For sequential code, acceleration and efficiency don't make sense
# as it should be the baseline (speedup = 1.0 by definition)
if task_type == "seq":
acceleration = "1.00" # Sequential is the baseline
efficiency = "N/A"
else:
acceleration = f"{speedup:.2f}"
efficiency = f"{speedup / eff_num_proc * 100:.2f}%"
except (ValueError, TypeError):
pass
return acceleration, efficiency
Expand Down Expand Up @@ -189,7 +195,7 @@ def main():

# Calculate acceleration and efficiency if performance data is available
acceleration, efficiency = calculate_performance_metrics(
perf_val, eff_num_proc
perf_val, eff_num_proc, task_type
)

# Calculate deadline penalty points
Expand Down
Loading