From 0fe9b68b5c0876619688bfe3c920be9d26837311 Mon Sep 17 00:00:00 2001 From: Leo Meyerovich Date: Thu, 17 Jul 2025 15:41:01 -0700 Subject: [PATCH 1/3] feat(ci): Update GPU CI workflow with environment protection and parallel testing MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Add gpu-staff environment for access control - Remove hardcoded user list (now handled by environment) - Enable parallel test execution with pytest-xdist (-n auto --dist loadfile) 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude --- .github/workflows/ci-gpu.yml | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/.github/workflows/ci-gpu.yml b/.github/workflows/ci-gpu.yml index c540780ca4..c114ac02ec 100644 --- a/.github/workflows/ci-gpu.yml +++ b/.github/workflows/ci-gpu.yml @@ -21,7 +21,7 @@ jobs: gpu-permission: if: > github.event_name == 'workflow_dispatch' || - (github.event_name == 'pull_request' && contains(fromJson('["lmeyerov", "tanmoyio", "aucahuasi", "silkspace", "DataBoyTx"]'), github.actor)) + github.event_name == 'pull_request' runs-on: ubuntu-latest steps: - name: Do nothing @@ -48,6 +48,7 @@ jobs: (github.event_name == 'pull_request' && contains(github.event.label.name, 'gpu-ci')) runs-on: group: GPU Runners - Public + environment: gpu-staff strategy: matrix: @@ -96,4 +97,4 @@ jobs: run: | source pygraphistry/bin/activate python -m pytest --version - python -B -m pytest -vv + python -B -m pytest -vv -n auto --dist loadfile From c70cc528be3f3dadfa9d1f210a993a2dbba5d27a Mon Sep 17 00:00:00 2001 From: Leo Meyerovich Date: Thu, 17 Jul 2025 15:56:21 -0700 Subject: [PATCH 2/3] fix(ci): Revert GPU CI environment changes, keep parallel testing MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Remove gpu-staff environment requirement - Restore hardcoded user list for GPU permissions - Keep parallel test execution improvement 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude --- .github/workflows/ci-gpu.yml | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/.github/workflows/ci-gpu.yml b/.github/workflows/ci-gpu.yml index c114ac02ec..e324aadee0 100644 --- a/.github/workflows/ci-gpu.yml +++ b/.github/workflows/ci-gpu.yml @@ -21,7 +21,7 @@ jobs: gpu-permission: if: > github.event_name == 'workflow_dispatch' || - github.event_name == 'pull_request' + (github.event_name == 'pull_request' && contains(fromJson('["lmeyerov", "tanmoyio", "aucahuasi", "silkspace", "DataBoyTx"]'), github.actor)) runs-on: ubuntu-latest steps: - name: Do nothing @@ -48,7 +48,6 @@ jobs: (github.event_name == 'pull_request' && contains(github.event.label.name, 'gpu-ci')) runs-on: group: GPU Runners - Public - environment: gpu-staff strategy: matrix: From 829cc84aa91cfe68a302df096cc45768b221d868 Mon Sep 17 00:00:00 2001 From: Leo Meyerovich Date: Thu, 17 Jul 2025 16:17:07 -0700 Subject: [PATCH 3/3] docs: Update CHANGELOG with GPU CI parallel testing improvement MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude --- CHANGELOG.md | 3 +++ 1 file changed, 3 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 790c7a9860..b7273b909e 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -10,6 +10,9 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm ### Fixed * Engine: Fix resolve_engine() to use dynamic import for Plottable isinstance check to avoid Jinja dependency from pandas df.style getter (#701) +### Infra +* CI: Enable parallel test execution in GPU CI with pytest-xdist (#703) + ### Docs * Update copyright year from 2024 to 2025 in documentation and LICENSE.txt * GFQL: Add comprehensive specification documentation (#698)