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

Add TRT-LLM backend build to Triton #3

Merged
merged 7 commits into from
Oct 4, 2023

Conversation

krishung5
Copy link

@krishung5 krishung5 commented Sep 29, 2023

CMakeLists.txt Outdated Show resolved Hide resolved
CMakeLists.txt Outdated
option(TRITON_ENABLE_STATS "Include statistics collections in backend" ON)

# The tags here should match the tag of base image in Dockerfile
set(TRITON_COMMON_REPO_TAG "r23.07" CACHE STRING "Tag for triton-inference-server/common repo")

Choose a reason for hiding this comment

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

Shouldn't the default value be set to "main"?

CMakeLists.txt Show resolved Hide resolved
tools/environment_setup.sh Outdated Show resolved Hide resolved
tools/environment_setup.sh Outdated Show resolved Hide resolved
tools/environment_setup.sh Outdated Show resolved Hide resolved
tools/gen_trtllm_dockerfile.py Outdated Show resolved Hide resolved
tools/gen_trtllm_dockerfile.py Show resolved Hide resolved
tools/gen_trtllm_dockerfile.py Outdated Show resolved Hide resolved
tools/gen_trtllm_dockerfile.py Show resolved Hide resolved
@krishung5 krishung5 requested a review from tanmayv25 October 3, 2023 04:58
Copy link

@tanmayv25 tanmayv25 left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@krishung5
Copy link
Author

Merging this PR. Will have a separate PR for cleaning files and adding examples/documentation.

@krishung5 krishung5 merged commit 1528789 into triton-trtllm-backend Oct 4, 2023
@krishung5 krishung5 deleted the krish-trtllm-backend-build branch October 4, 2023 00:03
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