Skip to content

Commit adfd9e0

Browse files
committed
ci: benchmark compiled (mypyc) and pure builds
Add a mypyc=true/false matrix axis to the benchmark workflow so the resolving benchmark runs for both the compiled and pure-python builds. mypyc requires Python >= 3.10, so the run moves from 3.7 to 3.10 and both variants run there for an apples-to-apples comparison. REZ_MYPYC is wired into the install step, and result artifacts are named per-variant. The store job keeps the pure (mypyc=false) result as the canonical baseline, since store_benchmark.py keys results by python+rez version only. Signed-off-by: Chad Dombrova <chadrik@gmail.com>
1 parent 9c09227 commit adfd9e0

1 file changed

Lines changed: 7 additions & 4 deletions

File tree

.github/workflows/benchmark.yaml

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,8 @@ jobs:
2121
strategy:
2222
matrix:
2323
python-version:
24-
- '3.7'
24+
- '3.10'
25+
mypyc: [false, true]
2526

2627
steps:
2728
- name: Checkout
@@ -33,6 +34,8 @@ jobs:
3334
python-version: ${{ matrix.python-version }}
3435

3536
- name: Install Rez
37+
env:
38+
REZ_MYPYC: ${{ matrix.mypyc && '1' || '0' }}
3639
run: |
3740
mkdir ./installdir
3841
@@ -51,7 +54,7 @@ jobs:
5154
5255
- uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1
5356
with:
54-
name: "benchmark-result-${{ matrix.python-version }}"
57+
name: "benchmark-result-${{ matrix.python-version }}-mypyc-${{ matrix.mypyc }}"
5558
path: ./out
5659

5760
store_benchmark_result:
@@ -62,15 +65,15 @@ jobs:
6265
strategy:
6366
matrix:
6467
python-version:
65-
- '3.7'
68+
- '3.10'
6669

6770
# so we don't have jobs trying to push to git at the same time
6871
max-parallel: 1
6972

7073
steps:
7174
- uses: actions/download-artifact@3e5f45b2cfb9172054b4087a40e8e0b5a5461e7c # v8.0.1
7275
with:
73-
name: "benchmark-result-${{ matrix.python-version }}"
76+
name: "benchmark-result-${{ matrix.python-version }}-mypyc-false"
7477
path: .
7578

7679
- name: Checkout (release)

0 commit comments

Comments
 (0)