Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] update tensorrt version to 8.5 #774

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

[WIP] update tensorrt version to 8.5 #774

wants to merge 3 commits into from

Conversation

zzpmiracle
Copy link
Collaborator

No description provided.

@chenbohua3
Copy link
Collaborator

LGTM! Thanks a lot!

@zzpmiracle zzpmiracle changed the title update tensorrt version to 8.5 [WIP] update tensorrt version to 8.5 Nov 24, 2022
@@ -18,7 +18,7 @@ jobs:
develop_base_image: nvidia/cuda:11.3.0-cudnn8-devel-ubuntu20.04
runtime_base_image: nvidia/cuda:11.3.0-cudnn8-devel-ubuntu20.04
extra_build_args: --build-arg PYTHON_VERSION=PYTHON3.8 --build-arg ENABLE_FIND_FASTEST_APT_SOURCE=OFF
extra_envs: -e TORCH_BLADE_BUILD_TENSORRT_STATIC=ON
extra_envs: -e TORCH_BLADE_BUILD_TENSORRT_STATIC=OFF
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm curious why?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Just for product test. Our product better to provide static linked version, to reduce user's envoiroment request. But we met some compile error for TRT8.5 like (.text._ZN5torch5blade21GetTorchMlirWhiteListEv+0x17): relocation truncated to fit: R_X86_64_PC32 against .bss._ZGVZN5torch5blade21GetTorchMlirWhiteListEvE10white_list' in all static link build. This used to occur in torch1.12+cu113 only, so change this to fix this remaining problem. I'm testing some compiler flag locally such as -fPIC.

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.

3 participants