Skip to content

Commit

Permalink
Replace Python 3.12 with Python 3.13 and add Python 3.10 to the matrix
Browse files Browse the repository at this point in the history
Expands test coverage to cover the oldest and newest Python versions that we support.

PiperOrigin-RevId: 721900273
  • Loading branch information
nitins17 authored and Google-ML-Automation committed Feb 4, 2025
1 parent 9f79799 commit 0625945
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 4 deletions.
12 changes: 10 additions & 2 deletions .github/workflows/bazel_cpu_rbe.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,14 +26,22 @@ jobs:
container: ${{ (contains(matrix.runner, 'linux-x86') && 'us-central1-docker.pkg.dev/tensorflow-sigs/tensorflow/ml-build:latest') ||
(contains(matrix.runner, 'linux-arm64') && 'us-central1-docker.pkg.dev/tensorflow-sigs/tensorflow/ml-build-arm64:latest') }}
env:
JAXCI_HERMETIC_PYTHON_VERSION: "3.12"
JAXCI_HERMETIC_PYTHON_VERSION: ${{ matrix.python }}
JAXCI_ENABLE_X64: ${{ matrix.enable-x_64 }}
# Begin Presubmit Naming Check - name modification requires internal check to be updated
strategy:
matrix:
python: ["3.10", "3.13"]
runner: ["linux-x86-n2-16", "linux-arm64-c4a-16"]
enable-x_64: [1, 0]
name: "Bazel CPU tests (${{ matrix.runner }}, Python 3.12, x64=${{ matrix.enable-x_64 }})"
exclude:
# Exclude x64=1 on the oldest Python and x64=0 on the newest Python. As long as we have
# coverage for one of each, we don't need to run both.
- python: "3.10"
enable-x_64: 1
- python: "3.13"
enable-x_64: 0
name: "Bazel CPU tests (${{ matrix.runner }}, Python ${{ matrix.python }}, x64=${{ matrix.enable-x_64 }})"
# End Presubmit Naming Check github-cpu-presubmits
steps:
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
Expand Down
12 changes: 10 additions & 2 deletions .github/workflows/bazel_cuda_rbe.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,14 +25,22 @@ jobs:
runs-on: ${{ matrix.runner }}
container: 'us-central1-docker.pkg.dev/tensorflow-sigs/tensorflow/ml-build:latest'
env:
JAXCI_HERMETIC_PYTHON_VERSION: "3.12"
JAXCI_HERMETIC_PYTHON_VERSION: ${{ matrix.python }}
JAXCI_ENABLE_X64: ${{ matrix.enable-x_64 }}
# Begin Presubmit Naming Check - name modification requires internal check to be updated
strategy:
matrix:
python: ["3.10", "3.13"]
runner: ["linux-x86-n2-16"]
enable-x_64: [1, 0]
name: "Bazel single accelerator CUDA tests (${{ matrix.runner }}, Python 3.12, x64=${{ matrix.enable-x_64 }})"
exclude:
# Exclude x64=1 on the oldest Python and x64=0 on the newest Python. As long as we have
# coverage for one of each, we don't need to run both.
- python: "3.10"
enable-x_64: 1
- python: "3.13"
enable-x_64: 0
name: "Bazel single accelerator CUDA tests (${{ matrix.runner }}, Python ${{ matrix.python }}, x64=${{ matrix.enable-x_64 }})"
# End Presubmit Naming Check github-cuda-presubmits
steps:
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
Expand Down

0 comments on commit 0625945

Please sign in to comment.