Skip to content

Commit

Permalink
memsuspend on acir_tests. don't overwrite noir-repo stuff.
Browse files Browse the repository at this point in the history
  • Loading branch information
@charlielye
charlielye committed Jan 8, 2025
1 parent 7e9e268 commit 137ade8
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions barretenberg/acir_tests/bootstrap.sh
View file
Original file line number Diff line number Diff line change
Expand Up @@ -21,9 +21,8 @@ function prepare_tests {
COMPILE=2 ./run_test.sh assert_statement
local bb=$(realpath ../cpp/build/bin/bb)
(cd ./acir_tests/assert_statement && \
$bb write_recursion_inputs_ultra_honk -b ./target/program.json -o ../../../../noir/noir-repo/test_programs/execution_success/verify_honk_proof --recursive && \
$bb write_recursion_inputs_rollup_honk -b ./target/program.json -o ../../../../noir/noir-repo/test_programs/execution_success/verify_rollup_honk_proof --recursive && \
cp -R ../../../../noir/noir-repo/test_programs/execution_success/verify_honk_proof .. && cp -R ../../../../noir/noir-repo/test_programs/execution_success/verify_rollup_honk_proof ..)
$bb write_recursion_inputs_ultra_honk -b ./target/program.json -o ../verify_honk_proof --recursive && \
$bb write_recursion_inputs_rollup_honk -b ./target/program.json -o ../verify_rollup_honk_proof --recursive)

# COMPILE=2 only compiles the test.
denoise "parallel --joblog joblog.txt --line-buffered 'COMPILE=2 ./run_test.sh \$(basename {})' ::: ./acir_tests/*"
Expand Down Expand Up @@ -74,7 +73,7 @@ function test {
# local jobs=$(($(nproc) / HARDWARE_CONCURRENCY))
local jobs=64

test_cmds | (cd $root; parallel -j$jobs --tag --line-buffered --joblog joblog.txt)
test_cmds | (cd $root; parallel --memsuspend 64G -j$jobs --tag --line-buffered --joblog joblog.txt)

cache_upload_flag barretenberg-acir-tests-$hash
github_endgroup
Expand Down

1 comment on commit 137ade8

@AztecBot
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'C++ Benchmark'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.05.

Benchmark suite Current: 137ade8 Previous: b8ef30e Ratio
commit(t) 3090299386 ns/iter 2860133484 ns/iter 1.08

This comment was automatically generated by workflow using github-action-benchmark.

CC: @ludamad @codygunton

Please sign in to comment.