Skip to content

Commit

Permalink
replace run-benchmark.py with cargo bench
Browse files Browse the repository at this point in the history
  • Loading branch information
arvidn committed Jun 21, 2023
1 parent 3f2c0df commit abe7d2c
Show file tree
Hide file tree
Showing 35 changed files with 673 additions and 423 deletions.
36 changes: 6 additions & 30 deletions .github/workflows/benchmark.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,37 +45,8 @@ jobs:
python -m pip install maturin
rustup target add x86_64-unknown-linux-musl
- name: Build Windows
if: startsWith(matrix.os, 'windows')
run: |
python -m venv venv
. .\venv\Scripts\Activate.ps1
ln -s venv\Scripts\Activate.ps1 activate
python -m pip install colorama
maturin develop -m wheel/Cargo.toml --release
- name: Run benchmarks (Windows)
if: startsWith(matrix.os, 'windows')
run: |
. .\venv\Scripts\Activate.ps1
python benchmark/run-benchmark.py
- name: Build
if: ${{ !startsWith(matrix.os, 'windows') }}
env:
CC: gcc
run: |
python -m venv venv
ln -s venv/bin/activate
. ./activate
python -m pip install colorama
maturin develop -m wheel/Cargo.toml --release --features=openssl
- name: Run benchmarks
if: ${{ !startsWith(matrix.os, 'windows') }}
run: |
. ./activate
python benchmark/run-benchmark.py
run: cargo bench

max-cost-checks:
name: Cost checks
Expand Down Expand Up @@ -105,6 +76,11 @@ jobs:
python -m pip install maturin
rustup target add x86_64-unknown-linux-musl
# we need this for brun and opc
- name: install clvm_tools
run: |
cargo install clvm_tools_rs
- name: Build
env:
CC: gcc
Expand Down
5 changes: 0 additions & 5 deletions .github/workflows/build-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -114,11 +114,6 @@ jobs:
python -m pip install pytest
python -m pip install blspy
# we need this for brun and opc
- name: install clvm_tools
run: |
cargo install clvm_tools_rs
- name: Run tests from wheel
run: |
. ./activate
Expand Down
Loading

0 comments on commit abe7d2c

Please sign in to comment.