Skip to content

Bump up nvComp version to 5.3.0.16#6428

Open
JanuszL wants to merge 1 commit into
NVIDIA:mainfrom
JanuszL:bump_up_nvcomp_version
Open

Bump up nvComp version to 5.3.0.16#6428
JanuszL wants to merge 1 commit into
NVIDIA:mainfrom
JanuszL:bump_up_nvcomp_version

Conversation

@JanuszL

@JanuszL JanuszL commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Category:

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

Description:

  • Bumps up nvComp version to 5.3.0.16

Additional information:

Affected modules and functionalities:

  • dockerfile with CUDA
  • CMake with dependencies

Key points relevant for the review:

  • NA

Tests:

  • Existing tests apply
    • test_inflate
  • 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: N/A

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

Copy link
Copy Markdown
Collaborator

CI MESSAGE: [59095008]: BUILD STARTED

@dali-automaton

Copy link
Copy Markdown
Collaborator

CI MESSAGE: [59095030]: BUILD STARTED

@greptile-apps

greptile-apps Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR bumps the nvComp dependency from 5.2.x to 5.3.0.16 across all build configurations. As a bonus cleanup, the cuda129 Dockerfiles now use ${NVCOMP_VERSION} consistently in the wget and cp commands (the old value 4.2.0.11-1 was set but never actually referenced), and the cuda133 Dockerfiles introduce the variable for the first time instead of hardcoding the version.

  • cmake/Dependencies.common.cmake: pip package pin updated to == 5.3.0.16, keeping it in sync with the tarball version used at build time.
  • All four Dockerfiles (cuda129/cuda133 × x86_64/aarch64): NVCOMP_VERSION set to 5.3.0.16 and used uniformly in the CDN download URL and extracted-archive cp paths.

Confidence Score: 5/5

Safe to merge — all five files change only version strings and the variable-parameterization is applied consistently across every platform Dockerfile and the cmake dependency pin.

The change is a straightforward version bump of one external library. The new version string 5.3.0.16 is applied uniformly in the CDN download URLs, the extracted-archive directory names, and the cmake pip-package requirement. A pre-existing inconsistency (old cuda129 NVCOMP_VERSION variable was declared but unused; cuda133 hardcoded the path) is also cleaned up here. No logic, API surface, or runtime behavior is altered.

No files require special attention.

Important Files Changed

Filename Overview
cmake/Dependencies.common.cmake Pip package pin for nvidia-libnvcomp updated from 5.2.0.13 to 5.3.0.16, now matching the tarball version used in all Dockerfiles.
docker/Dockerfile.cuda129.aarch64.deps NVCOMP_VERSION bumped from 4.2.0.11-1 to 5.3.0.16 and the variable is now actually used in the wget URL and cp commands (previously those lines were hardcoded to 5.2.0.10).
docker/Dockerfile.cuda129.x86_64.deps Same fix as the aarch64 variant: NVCOMP_VERSION updated to 5.3.0.16 and used consistently in wget and cp commands.
docker/Dockerfile.cuda133.aarch64.deps NVCOMP_VERSION=5.3.0.16 introduced as a variable (was previously hardcoded as 5.2.0.10) and used in wget and cp commands.
docker/Dockerfile.cuda133.x86_64.deps Same as the cuda133 aarch64 variant: NVCOMP_VERSION introduced and updated to 5.3.0.16.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Docker build] --> B{Platform}
    B --> C[cuda129 x86_64]
    B --> D[cuda129 aarch64]
    B --> E[cuda133 x86_64]
    B --> F[cuda133 aarch64]
    C --> G["wget nvcomp-linux-x86_64-5.3.0.16_cuda12-archive.tar.xz"]
    D --> H["wget nvcomp-linux-sbsa-5.3.0.16_cuda12-archive.tar.xz"]
    E --> I["wget nvcomp-linux-x86_64-5.3.0.16_cuda13-archive.tar.xz"]
    F --> J["wget nvcomp-linux-sbsa-5.3.0.16_cuda13-archive.tar.xz"]
    G & H & I & J --> K["Install headers + libs to /usr/local/cuda"]
    K --> L[DALI build]
    L --> M["cmake: nvidia-libnvcomp-cu{MAJOR} == 5.3.0.16 (pip requirement)"]
Loading

Reviews (1): Last reviewed commit: "Bump up nvComp version to 5.3.0.16" | Re-trigger Greptile

@dali-automaton

Copy link
Copy Markdown
Collaborator

CI MESSAGE: [59095030]: BUILD FAILED

@dali-automaton

Copy link
Copy Markdown
Collaborator

CI MESSAGE: [59095008]: BUILD FAILED

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants