Skip to content

Commit

Permalink
Merge branch 'pola-rs:main' into main
Browse files Browse the repository at this point in the history
  • Loading branch information
Niko32 authored Oct 30, 2024
2 parents 28e7fc7 + 569d5b7 commit fb81232
Show file tree
Hide file tree
Showing 794 changed files with 19,537 additions and 10,888 deletions.
5 changes: 3 additions & 2 deletions .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
/crates/polars-sql/ @ritchie46 @orlp @c-peters @alexander-beedie
/crates/polars-parquet/ @ritchie46 @orlp @c-peters @coastalwhite
/crates/polars-time/ @ritchie46 @orlp @c-peters @MarcoGorelli
/crates/polars-python/ @ritchie46 @c-peters @alexander-beedie @MarcoGorelli @reswqa
/crates/polars-python/src/lazyframe/visit.rs @ritchie46 @c-peters @alexander-beedie @MarcoGorelli @reswqa @wence-
/crates/polars-python/src/lazyframe/visitor/ @ritchie46 @c-peters @alexander-beedie @MarcoGorelli @reswqa @wence-
/py-polars/ @ritchie46 @c-peters @alexander-beedie @MarcoGorelli @reswqa
/py-polars/src/lazyframe/visit.rs @ritchie46 @c-peters @alexander-beedie @MarcoGorelli @reswqa @wence-
/py-polars/src/lazyframe/visitor/ @ritchie46 @c-peters @alexander-beedie @MarcoGorelli @reswqa @wence-
16 changes: 13 additions & 3 deletions .github/workflows/benchmark-remote.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,19 +2,22 @@ name: Remote Benchmark

on:
workflow_dispatch:
push:
branches:
- 'main'
pull_request:
types: [ labeled ]

concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
cancel-in-progress: true
cancel-in-progress: ${{ github.event.label.name == 'needs-bench' }}

env:
SCALE_FACTOR: '10.0'

jobs:
main:
if: ${{ github.event.label.name == 'needs-bench' }}
if: ${{ github.ref == 'refs/heads/main' || github.event.label.name == 'needs-bench' }}
runs-on: self-hosted
steps:
- uses: actions/checkout@v4
Expand Down Expand Up @@ -62,4 +65,11 @@ jobs:
- name: Run benchmark
working-directory: polars-benchmark
run: |
make run-polars-no-env
make run-polars-no-env | tee ../py-polars/benchmark-results
- name: Cache the Polars build
if: ${{ github.ref == 'refs/heads/main' }}
working-directory: py-polars
run: |
"$HOME/py-polars-cache/save_benchmark_data.py" "$PWD/polars" < ./benchmark-results
"$HOME/py-polars-cache/cache-build.sh" "$PWD/polars"
2 changes: 1 addition & 1 deletion .github/workflows/benchmark.yml
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ jobs:
env:
RUSTFLAGS: -C embed-bitcode -D warnings
working-directory: py-polars
run: maturin develop --features new_streaming --release -- -C codegen-units=8 -C lto=thin -C target-cpu=native
run: maturin develop --release -- -C codegen-units=8 -C lto=thin -C target-cpu=native

- name: Run benchmark tests
uses: CodSpeedHQ/action@v3
Expand Down
5 changes: 2 additions & 3 deletions .github/workflows/release-python.yml
Original file line number Diff line number Diff line change
Expand Up @@ -137,7 +137,6 @@ jobs:
if: matrix.architecture == 'x86-64'
env:
IS_LTS_CPU: ${{ matrix.package == 'polars-lts-cpu' }}
IS_MACOS: ${{ matrix.os == 'macos-13' }}
# IMPORTANT: All features enabled here should also be included in py-polars/polars/_cpu_check.py
run: |
if [[ "$IS_LTS_CPU" = true ]]; then
Expand Down Expand Up @@ -175,7 +174,7 @@ jobs:
- name: Set variables in CPU check module - LTS_CPU
if: matrix.package == 'polars-lts-cpu'
run: |
sed $SED_INPLACE 's/^_LTS_CPU = False$/_LTS_CPU = True/g' $CPU_CHECK_MODULE
sed $SED_INPLACE 's/^_POLARS_LTS_CPU = False$/_POLARS_LTS_CPU = True/g' $CPU_CHECK_MODULE
- name: Set Rust target for aarch64
if: matrix.architecture == 'aarch64'
Expand All @@ -195,7 +194,7 @@ jobs:
command: build
target: ${{ steps.target.outputs.target }}
args: >
--release
--profile dist-release
--manifest-path py-polars/Cargo.toml
--out dist
manylinux: ${{ matrix.architecture == 'aarch64' && '2_24' || 'auto' }}
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test-python.yml
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ jobs:
save-if: ${{ github.ref_name == 'main' }}

- name: Install Polars
run: maturin develop --features new_streaming
run: maturin develop

- name: Run doctests
if: github.ref_name != 'main' && matrix.python-version == '3.12' && matrix.os == 'ubuntu-latest'
Expand Down
Loading

0 comments on commit fb81232

Please sign in to comment.