Skip to content

Conversation

@MartinMarciniszyn
Copy link
Collaborator

Update Quick Start Guide

Add explanations to the quick start guide for running the examples within the NGC release container.

@MartinMarciniszyn MartinMarciniszyn requested a review from a team as a code owner June 18, 2025 13:42
Signed-off-by: Martin Marciniszyn Mehringer <[email protected]>
@laikhtewari
Copy link
Collaborator

The easiest (ie lightest weight) way would be with pip

We consistently hear from devs (both internal SAs and external users) that they prefer pip when quickly testing new libs

Pip should be highlighted here first in addition to ngc container I think

@MartinMarciniszyn
Copy link
Collaborator Author

The easiest (ie lightest weight) way would be with pip

We consistently hear from devs (both internal SAs and external users) that they prefer pip when quickly testing new libs

Pip should be highlighted here first in addition to ngc container I think

I agree that pip would be the easiest option in theory. However, our binary dependencies are so involved that pip install rarely works on a bare metal system. We would need to describe how to start a docker container in which one can run pip install. And that seems more complicated than starting the container with TRT-LLM pre-installed in the first place.

@MartinMarciniszyn
Copy link
Collaborator Author

/bot skip --comment "documentation change"

@MartinMarciniszyn MartinMarciniszyn enabled auto-merge (squash) June 19, 2025 07:30
@tensorrt-cicd
Copy link
Collaborator

PR_Github #9472 [ skip ] triggered by Bot

@tensorrt-cicd
Copy link
Collaborator

PR_Github #9472 [ skip ] completed with state SUCCESS
Skipping testing for commit decd2f4

@MartinMarciniszyn MartinMarciniszyn merged commit 47a765d into NVIDIA:release/0.20 Jun 19, 2025
3 checks passed
MartinMarciniszyn added a commit to MartinMarciniszyn/TensorRT-LLM that referenced this pull request Jun 19, 2025
Signed-off-by: Martin Marciniszyn Mehringer <[email protected]>
MartinMarciniszyn added a commit that referenced this pull request Jun 19, 2025
Signed-off-by: Martin Marciniszyn Mehringer <[email protected]>
dc3671 pushed a commit to dc3671/TensorRT-LLM that referenced this pull request Jul 1, 2025
Signed-off-by: Martin Marciniszyn Mehringer <[email protected]>
dc3671 pushed a commit to dc3671/TensorRT-LLM that referenced this pull request Jul 1, 2025
Signed-off-by: Martin Marciniszyn Mehringer <[email protected]>
dc3671 pushed a commit to dc3671/TensorRT-LLM that referenced this pull request Jul 1, 2025
Signed-off-by: Martin Marciniszyn Mehringer <[email protected]>
dc3671 pushed a commit that referenced this pull request Jul 1, 2025
Signed-off-by: Martin Marciniszyn Mehringer <[email protected]>
Shunkangz pushed a commit to Shunkangz/TensorRT-LLM that referenced this pull request Jul 2, 2025
Signed-off-by: Martin Marciniszyn Mehringer <[email protected]>
dominicshanshan pushed a commit to dominicshanshan/TensorRT-LLM that referenced this pull request Jul 9, 2025
Signed-off-by: Martin Marciniszyn Mehringer <[email protected]>
dominicshanshan pushed a commit to dominicshanshan/TensorRT-LLM that referenced this pull request Jul 10, 2025
Signed-off-by: Martin Marciniszyn Mehringer <[email protected]>
dominicshanshan pushed a commit to dominicshanshan/TensorRT-LLM that referenced this pull request Jul 10, 2025
Signed-off-by: Martin Marciniszyn Mehringer <[email protected]>
amirkl94 pushed a commit to amirkl94/TensorRT-LLM that referenced this pull request Jul 10, 2025
amirkl94 pushed a commit to amirkl94/TensorRT-LLM that referenced this pull request Jul 10, 2025
dominicshanshan pushed a commit to dominicshanshan/TensorRT-LLM that referenced this pull request Jul 10, 2025
Signed-off-by: Martin Marciniszyn Mehringer <[email protected]>
dominicshanshan pushed a commit to dominicshanshan/TensorRT-LLM that referenced this pull request Jul 10, 2025
Signed-off-by: Martin Marciniszyn Mehringer <[email protected]>
@MartinMarciniszyn MartinMarciniszyn deleted the user/martinma/doc_quick_start_container branch August 12, 2025 08:17
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.

4 participants