From 294a5e461695edbd2eaac0757c207ec881beedf3 Mon Sep 17 00:00:00 2001 From: Guillaume Lagrange Date: Fri, 24 Apr 2026 15:28:51 +0200 Subject: [PATCH 1/2] feat: only wrap walltime command with sudo on linux We do not yet need sudo, since we don't use launchtl nor profilers yet. --- src/executor/wall_time/executor.rs | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/executor/wall_time/executor.rs b/src/executor/wall_time/executor.rs index 4da11899..a8e17934 100644 --- a/src/executor/wall_time/executor.rs +++ b/src/executor/wall_time/executor.rs @@ -163,7 +163,12 @@ impl Executor for WallTimeExecutor { ) .await } else { - let cmd = wrap_with_sudo(cmd_builder)?.build(); + let cmd_builder = if cfg!(target_os = "linux") { + wrap_with_sudo(cmd_builder)? + } else { + cmd_builder + }; + let cmd = cmd_builder.build(); debug!("cmd: {cmd:?}"); run_command_with_log_pipe(cmd).await } From bd91957b3f99f1133707af2aa206c5bb8a64ecc4 Mon Sep 17 00:00:00 2001 From: Guillaume Lagrange Date: Fri, 24 Apr 2026 15:38:43 +0200 Subject: [PATCH 2/2] feat: display N/A rather than NaN/inf in the benchmarks results table --- src/upload/benchmark_display.rs | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/upload/benchmark_display.rs b/src/upload/benchmark_display.rs index 8d1586f5..0dbdff6d 100644 --- a/src/upload/benchmark_display.rs +++ b/src/upload/benchmark_display.rs @@ -29,6 +29,9 @@ fn format_with_thousands_sep(n: u64) -> String { /// Format StdDev with color coding based on value fn format_stdev_colored(stdev_pct: f64) -> String { + if !stdev_pct.is_finite() { + return format!("{}", style("N/A").dim()); + } let formatted = format!("{stdev_pct:.2}%"); if stdev_pct <= 2.0 { format!("{}", style(&formatted).green())