Skip to content

Conversation

@liulehui
Copy link
Contributor

Description

  1. Jax dependency is introduced in
    [train][jax] Enable Jax trainer on GPU #58322
  2. The current test environment is for CUDA 12.1, which limit jax
    version below 0.4.14.
  3. jax <= 0.4.14 does not support py 3.12.
  4. skip jax test if it runs against py3.12+.

Signed-off-by: Lehui Liu <lehui@anyscale.com>
@liulehui liulehui requested a review from a team as a code owner January 12, 2026 17:16
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

The pull request correctly implements skipping JAX-related tests for Python 3.12+ environments. This is a necessary change to address the current incompatibility between older JAX versions (constrained by CUDA 12.1) and Python 3.12+. The use of @pytest.mark.skipif with a clear reason is appropriate and ensures the test suite remains functional on newer Python versions while awaiting updated JAX support. The changes are consistent across all affected tests.

Signed-off-by: Lehui Liu <lehui@anyscale.com>
@ray-gardener ray-gardener bot added the train Ray Train Related Issue label Jan 12, 2026
@liulehui liulehui added the go add ONLY when ready to merge, run all tests label Jan 12, 2026
@matthewdeng matthewdeng merged commit 2b91eb6 into ray-project:master Jan 13, 2026
7 checks passed
rushikeshadhav pushed a commit to rushikeshadhav/ray that referenced this pull request Jan 14, 2026
## Description
1. Jax dependency is introduced in
ray-project#58322
2. The current test environment is for CUDA 12.1, which limit jax
version below 0.4.14.
3. jax <= 0.4.14 does not support py 3.12.
4. skip jax test if it runs against py3.12+.

---------

Signed-off-by: Lehui Liu <lehui@anyscale.com>
jeffery4011 pushed a commit to jeffery4011/ray that referenced this pull request Jan 20, 2026
## Description
1. Jax dependency is introduced in
ray-project#58322
2. The current test environment is for CUDA 12.1, which limit jax
version below 0.4.14.
3. jax <= 0.4.14 does not support py 3.12.
4. skip jax test if it runs against py3.12+.

---------

Signed-off-by: Lehui Liu <lehui@anyscale.com>
Signed-off-by: jeffery4011 <jefferyshen1015@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

go add ONLY when ready to merge, run all tests train Ray Train Related Issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants