Skip to content

Commit

Permalink
ci: Don't include debug symbols in benchmark run (#20571)
Browse files Browse the repository at this point in the history
  • Loading branch information
ritchie46 authored Jan 10, 2025
1 parent 17556e4 commit c4b704b
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 6 deletions.
13 changes: 7 additions & 6 deletions .github/workflows/benchmark.yml
Original file line number Diff line number Diff line change
Expand Up @@ -67,15 +67,16 @@ jobs:
- name: Run sccache-cache
uses: mozilla-actions/sccache-action@v0.0.4

- name: Install Polars release build
env:
RUSTFLAGS: -C embed-bitcode -D warnings
- name: Build Polars release build
working-directory: py-polars
run: maturin develop --release -- -C codegen-units=8 -C lto=thin -C target-cpu=native
run: maturin build --profile nodebug-release -- -C codegen-units=8 -C lto=thin -C target-cpu=native

- name: Install Polars release build
run: pip install target/wheels/polars*.whl

- name: Set lib size
- name: Set wheel size
run: |
LIB_SIZE=$(ls -l py-polars/polars/polars*.so | awk '{ print $5 }')
LIB_SIZE=$(ls -l target/wheels/polars*.whl | awk '{ print $5 }')
echo "LIB_SIZE=$LIB_SIZE" >> $GITHUB_ENV
- name: Comment wheel size
Expand Down
1 change: 1 addition & 0 deletions py-polars/tests/unit/test_polars_import.py
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,7 @@ def _import_timings_as_frame(n_tries: int) -> tuple[pl.DataFrame, int]:


@pytest.mark.skipif(sys.platform == "win32", reason="Unreliable on Windows")
@pytest.mark.debug
@pytest.mark.slow
def test_polars_import() -> None:
# up-front compile '.py' -> '.pyc' before timing
Expand Down

0 comments on commit c4b704b

Please sign in to comment.