Skip to content

Commit

Permalink
chore(levm): ci: compare against new benchmarks in main (#1834)
Browse files Browse the repository at this point in the history
**Motivation**

Since the recently #1803 PR we now have new benchmarks. We need to
modify the CI to make the comparison.
I couldn't do that in the previous PR because since the new benchmarks
were not in main yet, the CI would always fail.

**Description**
I will fix this in the next benchmark PR so we don't have to create 2
PRs for each new benchmark.
<!-- A clear and concise general description of the changes this PR
introduces -->

<!-- Link to issues: Resolves #111, Resolves #222 -->
  • Loading branch information
dsocolobsky authored Jan 29, 2025
1 parent d085ac0 commit 10fee17
Showing 1 changed file with 19 additions and 9 deletions.
28 changes: 19 additions & 9 deletions .github/workflows/ci_bench_levm_in_pr.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -38,19 +38,19 @@ jobs:
make revm-comparison-ci
{
echo "#### Benchmark Results: Factorial";
cat Factorial.md;
cat Factorial.md || echo "No results";
echo "#### Benchmark Results: Factorial - Recursive";
cat FactorialRecursive.md;
cat FactorialRecursive.md || echo "No results";
echo "#### Benchmark Results: Fibonacci";
cat Fibonacci.md;
cat Fibonacci.md || echo "No results";
echo "#### Benchmark Results: ManyHashes";
cat ManyHashes.md;
cat ManyHashes.md || echo "No results";
echo "#### Benchmark Results: ERC20 - Transfer";
cat ERC20Transfer.md;
cat ERC20Transfer.md || echo "No results";
echo "#### Benchmark Results: ERC20 - Mint";
cat ERC20Mint.md;
cat ERC20Mint.md || echo "No results";
echo "#### Benchmark Results: ERC20 - Approval";
cat ERC20Approval.md;
cat ERC20Approval.md || echo "No results";
} > pr_result.md
- name: Upload PR results
Expand Down Expand Up @@ -90,9 +90,19 @@ jobs:
make revm-comparison-ci
{
echo "#### Benchmark Results: Factorial";
cat factorial.md;
cat Factorial.md || echo "No results";
echo "#### Benchmark Results: Factorial - Recursive";
cat FactorialRecursive.md || echo "No results";
echo "#### Benchmark Results: Fibonacci";
cat fibonacci.md;
cat Fibonacci.md || echo "No results";
echo "#### Benchmark Results: ManyHashes";
cat ManyHashes.md || echo "No results";
echo "#### Benchmark Results: ERC20 - Transfer";
cat ERC20Transfer.md || echo "No results";
echo "#### Benchmark Results: ERC20 - Mint";
cat ERC20Mint.md || echo "No results";
echo "#### Benchmark Results: ERC20 - Approval";
cat ERC20Approval.md || echo "No results";
} > main_result.md
- name: Upload main results
Expand Down

0 comments on commit 10fee17

Please sign in to comment.