🐛 [Bug] The version for CUDA 11 depends on CUDA 12 libraries

[bryant1410]

Bug Description

The version of this package for CUDA 11.8 depends on CUDA 12 libraries:

TensorRT/pyproject.toml

Lines 46 to 47 in 4aa6e79

	"tensorrt-cu12_bindings==10.1.0",
	"tensorrt-cu12_libs==10.1.0",

This can cause issues on a system that doesn't support CUDA 12, because this or another library can pick up on them.

To Reproduce

You can download, for example, https://download.pytorch.org/whl/cu118/torch_tensorrt-2.4.0%2Bcu118-cp310-cp310-linux_x86_64.whl and inspect its dependencies.

Expected behavior

I guess the expected behavior would be to depend on the exact same dependencies but for CUDA 11.

Environment

Build information about Torch-TensorRT can be found by turning on debug messages

• Torch-TensorRT Version (e.g. 1.0.0): 2.4.0
• PyTorch Version (e.g. 1.0): 2.4.0
• CPU Architecture: Intel
• OS (e.g., Linux): Linux
• How you installed PyTorch (conda, pip, libtorch, source): pip
• Build command you used (if compiling from source):
• Are you using local sources or building from archives:
• Python version: 3.10
• CUDA version: 11.8
• GPU models and configuration: A10
• Any other relevant information:

[bryant1410]

And, if I understand correctly, that version should also depend on tensorrt-cu11 and not on tensorrt.

[lanluo-nvidia]

@bryant1410
if you download tensorrt-10.1.0 wheel from pypi and you will see the requires.txt has tensorrt-cu12 regardless cuda version is cu121 or cu118.
hence as long as we have tensorrt-10.1.0 as dependency it will always download tensorrt-cu12 libraries.

What I did in the PR: #3105 is to make sure torch_tensorrt dependency in the pyproject.toml is correct.
From the pipeline result, I can see this will download both tensorrt-cu11-10.1.0 and tensorrt-cu12-10.2.0 as dependency for cu118 version.

[bryant1410]

shouldn't the CUDA 11 of this library (torch-tensorrt) depend on only tensorrt-cu11 (as opposed to also tensorrt)?

The problem with having both is that I have experienced my CUDA-11 PyTorch installation picking up on the incorrect CUDA library for some stuff, causing issues. My workaround so far is to skip installing Torch-TensorRT.

[bryant1410]

TensorRT Python meta-package currently defaults to CUDA 12, as per the docs:

You can append -cu11 or -cu12 to any Python module if you require a different CUDA major version. When unspecified, the TensorRT Python meta-packages default to the CUDA 12.x variants, the latest CUDA version supported by TensorRT.

Source: https://docs.nvidia.com/deeplearning/tensorrt/install-guide/index.html#installing-pip

This may not be great for future compatibility (though I don't have a strong opinion). Especially since they seem to change the default over time (e.g., see the release notes for TensorRT v10.0.1, where they changed the default to CUDA 12 for the first time if I understand correctly -- based on the previous release notes too).

It seems to me then that the best and official solution is to make the the CUDA 11 version depend only on tensorrt-cu11.

[lanluo-nvidia]

@bryant1410 thanks for your input:
with this fix #3105
I have verified it is now only installing cu11 related dependencies for cu118:
Successfully installed nvidia-cuda-runtime-cu11-2022.4.25 nvidia-cuda-runtime-cu117-11.7.60 tensorrt-cu11-10.1.0 tensorrt-cu11-bindings-10.1.0 tensorrt-cu11-libs-10.1.0 torch-tensorrt-2.5.0.dev20240820+cu118

[bryant1410]

Great!