Skip to content

Move to CUDA 13.3#6371

Open
JanuszL wants to merge 1 commit into
NVIDIA:mainfrom
JanuszL:cuda_13.3
Open

Move to CUDA 13.3#6371
JanuszL wants to merge 1 commit into
NVIDIA:mainfrom
JanuszL:cuda_13.3

Conversation

@JanuszL
Copy link
Copy Markdown
Contributor

@JanuszL JanuszL commented May 26, 2026

Category:

Other (e.g. Documentation, Tests, Configuration)

Description:

  • Moves to CUDA 13.3

Additional information:

Affected modules and functionalities:

  • docker toolchain files
  • docs

Key points relevant for the review:

  • NA

Tests:

  • Existing tests apply
    • build for CUDA 13.3
  • New tests added
    • Python tests
    • GTests
    • Benchmark
    • Other
  • N/A

Checklist

Documentation

  • Existing documentation applies
  • Documentation updated
    • Docstring
    • Doxygen
    • RST
    • Jupyter
    • Other
  • N/A

DALI team only

Requirements

  • Implements new requirements
  • Affects existing requirements
  • N/A

REQ IDs: N/A

JIRA TASK: DALI-4702

Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
@dali-automaton
Copy link
Copy Markdown
Collaborator

CI MESSAGE: [52687833]: BUILD STARTED

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 26, 2026

Greptile Summary

This PR adds CUDA 13.3 as the new default build target for DALI. Two new Dockerfile dependency files are introduced (x86_64 and aarch64), build.sh is updated to accept and default to CUDA 13.3, and the compilation docs are updated to reflect 13.3 as the officially supported version with 13.2 moving to the deprecated list.

  • New Dockerfile.cuda133.{x86_64,aarch64}.deps files mirror the 13.2 pattern with CUDA_SUBVERSION=13.3.0-1, CUFILE_VERSION=1.18.0.66-1, and CUDA_VERSION_MINOR=3; nvcomp reuses the same cuda13 archive as 13.2.
  • build.sh validation allow-list, default version, and error message are all updated consistently; docs now lists 13.3 as the supported default and 13.2 as deprecated.

Confidence Score: 5/5

Straightforward toolchain version bump with no logic changes; all four files are consistent with each other and with the established 13.2 pattern.

The new Dockerfiles are faithful copies of their 13.2 counterparts with the correct version strings updated. The build.sh allow-list, default, and error message are all updated in lockstep. The docs correctly promote 13.3 and demote 13.2. No regressions are introduced in any shared code paths.

No files require special attention.

Important Files Changed

Filename Overview
docker/Dockerfile.cuda133.x86_64.deps New CUDA 13.3 dependency Dockerfile for x86_64; mirrors the 13.2 file with updated CUDA_SUBVERSION (13.3.0-1), CUFILE_VERSION (1.18.0.66-1), and CUDA_VERSION_MINOR (3).
docker/Dockerfile.cuda133.aarch64.deps New CUDA 13.3 dependency Dockerfile for aarch64; mirrors the 13.2 file with the same version bumps.
docker/build.sh Default CUDA_VERSION bumped to 13.3, '133' added to the validation allow-list, and the help text and error message updated consistently. No logic changes.
docs/compilation.rst CUDA version references updated: 13.3 is now the officially supported default, 13.2 moved to the deprecated list.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[build.sh] -->|CUDA_VERSION=13.3| B{Validation}
    B -->|valid| C[Dockerfile.cuda133.x86_64.deps\nor .aarch64.deps]
    B -->|invalid| D[Error & exit]
    C --> E[Install CUDA 13.3 packages\ncuda-minimal-build, nvml, npp, etc.]
    E --> F[Install nvJPEG2K 0.9.0.43]
    F --> G[Install cuFile 1.18.0.66]
    G --> H[Download nvcomp 5.2.0.10_cuda13]
    H --> I[Install fatbinary wrapper\nfor clang compatibility]
    I --> J[CUDA 13.3 build image ready]
Loading

Reviews (1): Last reviewed commit: "Move to CUDA 13.3" | Re-trigger Greptile

@dali-automaton
Copy link
Copy Markdown
Collaborator

CI MESSAGE: [52687833]: BUILD FAILED

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants