bench table fix

Author: mohdsaid497566

toolchain/mfc/bench.py

@@ -111,13 +111,13 @@ def diff():
     table.add_column("[bold]Pre Process[/bold]", justify="right")
     table.add_column("[bold]Simulation[/bold]", justify="right")
     table.add_column("[bold]Post Process[/bold]", justify="right")
-    
+
     err = 0
     for slug in slugs:
         lhs_summary, rhs_summary = lhs["cases"][slug]["output_summary"], rhs["cases"][slug]["output_summary"]
-        speedups = []
-
-        for target in sorted(DEFAULT_TARGETS, key=lambda t: t.runOrder):
+        speedups = ['N/A', 'N/A', 'N/A']
+        
+        for i, target in enumerate(sorted(DEFAULT_TARGETS, key=lambda t: t.runOrder)):
             if (target.name not in lhs_summary) or (target.name not in rhs_summary):
                 cons.print(f"{target.name} not present in lhs_summary or rhs_summary - Case: {slug}")
                 err = 1; continue
@@ -126,24 +126,27 @@ def diff():
                 err = 1
                 cons.print(f"lhs_summary or rhs_summary reports non-real exec time for {target.name} - Case: {slug}")
 
+            exec_time_speedup = "N/A"
             try:
                 exec_time_value = lhs_summary[target.name]["exec"] / rhs_summary[target.name]["exec"]
-                speedups = f"Exec: {exec_time_value:.2f}"
+                exec_time_speedup = f"Exec: {exec_time_value:.2f}"
                 if exec_time_value < 0.9:
                     cons.print(f"[bold yellow]Warning[/bold yellow]: Exec time speedup for {target.name} is less than 0.9 - Case: {slug}")
-
+                speedups[i] = f"Exec: {exec_time_speedup}"
                 if target == SIMULATION:
+                    grind_time_speedup = "N/A"
                     if not math.isfinite(lhs_summary[target.name]["grind"]) or not math.isfinite(rhs_summary[target.name]["grind"]):
                         err = 1
                         cons.print(f"lhs_summary or rhs_summary reports non-real grind time for {target.name} - Case: {slug}")
 
                     grind_time_value = lhs_summary[target.name]["grind"] / rhs_summary[target.name]["grind"]
-                    speedups += f" & Grind: {grind_time_value:.2f}"
+                    grind_time_speedup += f" & Grind: {grind_time_value:.2f}"
                     if grind_time_value <0.98:
                         raise MFCException(f"Benchmarking failed since grind time speedup for {target.name} below acceptable threshold (<0.98) - Case: {slug}")
             except Exception as _:
                 err = 1
                 cons.print(f"lhs_summary or rhs_summary reports non-real grind time for {target.name} - Case: {slug}")
+            speedups[i] += f" & Grind: {grind_time_speedup}"
 
         table.add_row(f"[magenta]{slug}[/magenta]", *speedups)