Skip to content

Commit

Permalink
remove CPUSET due to a it not working as we see a lot of NaN in the r…
Browse files Browse the repository at this point in the history
…esults (#161)
  • Loading branch information
rluvaton authored Apr 10, 2024
1 parent a008e84 commit f945bb5
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions benchmarks/community-benchmark/run.sh
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,6 @@ echo "The following are optional across both use cases"
echo "RUNS = defaults to empty"
echo "FILTER = defaults to empty"
echo "MACHINE_THREADS - used for building node. Defaults to all threads on machine"
echo "CPUSET - used for pinning to specific CPU cores. Default to 0-11 (performance cores)"
}

if [ -z $PULL_ID ]; then
Expand All @@ -54,7 +53,6 @@ fi
mandatory CATEGORY
optional RUNS
optional FILTER
optional CPUSET "0-11"
getMACHINE_THREADS=`cat /proc/cpuinfo |grep processor|tail -n1|awk {'print $3'}`
let getMACHINE_THREADS=getMACHINE_THREADS+1 #getting threads this way is 0 based. Add one
optional MACHINE_THREADS $getMACHINE_THREADS
Expand Down Expand Up @@ -98,14 +96,11 @@ fi
if [ -n "$RUNS" ]; then
RUNS="--runs ${RUNS}"
fi
if [ -n "$CPUSET" ]; then
CPUSET="--set CPUSET=${CPUSET}"
fi
# run benchmark
fileName=output`date +%d%m%y-%H%M%S`.csv
echo "Output will be saved to $fileName"
pwd
./node-master benchmark/compare.js $CPUSET --old ./node-master --new ./node-pr $FILTER $RUNS -- $CATEGORY | tee $fileName
./node-master benchmark/compare.js --old ./node-master --new ./node-pr $FILTER $RUNS -- $CATEGORY | tee $fileName

cat $fileName | Rscript benchmark/compare.R
mv $fileName $startDir

0 comments on commit f945bb5

Please sign in to comment.