From 7d39df0aabd2a3208d94df882a3cce205ee7cc0e Mon Sep 17 00:00:00 2001 From: BryanFRD Date: Thu, 2 Jul 2026 23:10:32 +0200 Subject: [PATCH] fix(bench): measure ferrflow_parallel only from the binary method --- scripts/run.sh | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/scripts/run.sh b/scripts/run.sh index fb03117..31fc0e6 100755 --- a/scripts/run.sh +++ b/scripts/run.sh @@ -319,10 +319,15 @@ for tool in $TOOLS; do # Competitors are single-threaded; pin ferrflow to one thread so the # comparison is reproducible and apples-to-apples. par_cmd keeps the - # all-cores variant for the separate parallel stat. + # all-cores variant for the separate parallel stat — binary method + # only, since the parallel result file is not method-scoped and the + # npm (npx) and docker methods would overwrite it with node/container + # startup overhead instead of the binary's all-cores time. par_cmd="" if [[ "$tool" == "ferrflow" ]]; then - par_cmd="$full_cmd" + if [[ "$method" == "binary" ]]; then + par_cmd="$full_cmd" + fi full_cmd="$tool_cmd --jobs 1 $cmd" fi