TensorRT-LLM icon indicating copy to clipboard operation
TensorRT-LLM copied to clipboard

[https://nvbugspro.nvidia.com/bug/5243740][fix] deduce default max_tokens for trtllm-serve

Open LinPoly opened this issue 7 months ago • 24 comments

Description

Deduce the default max_tokens for both TensorRT and PyTorch workflow, to improve usability, remove the default value of max_tokens in openai_protocol.py.

Test Coverage

Original tests are good.

GitHub Bot Help

/bot [-h] ['run', 'kill', 'skip', 'reuse-pipeline'] ...

Provide a user friendly way for developers to interact with a Jenkins server.

Run /bot [-h|--help] to print this help message.

See details below for each supported subcommand.

run [--disable-fail-fast --skip-test --stage-list "A10-1, xxx" --gpu-type "A30, H100_PCIe" --add-multi-gpu-test --only-multi-gpu-test --disable-multi-gpu-test --post-merge --extra-stage "H100_PCIe-[Post-Merge]-1, xxx"]

Launch build/test pipelines. All previously running jobs will be killed.

--disable-fail-fast (OPTIONAL) : Disable fail fast on build/tests/infra failures.

--skip-test (OPTIONAL) : Skip all test stages, but still run build stages, package stages and sanity check stages. Note: Does NOT update GitHub check status.

--stage-list "A10-1, xxx" (OPTIONAL) : Only run the specified test stages. Examples: "A10-1, xxx". Note: Does NOT update GitHub check status.

--gpu-type "A30, H100_PCIe" (OPTIONAL) : Only run the test stages on the specified GPU types. Examples: "A30, H100_PCIe". Note: Does NOT update GitHub check status.

--only-multi-gpu-test (OPTIONAL) : Only run the multi-GPU tests. Note: Does NOT update GitHub check status.

--disable-multi-gpu-test (OPTIONAL) : Disable the multi-GPU tests. Note: Does NOT update GitHub check status.

--add-multi-gpu-test (OPTIONAL) : Force run the multi-GPU tests. Will also run L0 pre-merge pipeline.

--post-merge (OPTIONAL) : Run the L0 post-merge pipeline instead of the ordinary L0 pre-merge pipeline.

--extra-stage "H100_PCIe-[Post-Merge]-1, xxx" (OPTIONAL) : Run the ordinary L0 pre-merge pipeline and specified test stages. Examples: --extra-stage "H100_PCIe-[Post-Merge]-1, xxx".

kill

kill

Kill all running builds associated with pull request.

skip

skip --comment COMMENT

Skip testing for latest commit on pull request. --comment "Reason for skipping build/test" is required. IMPORTANT NOTE: This is dangerous since lack of user care and validation can cause top of tree to break.

reuse-pipeline

reuse-pipeline

Reuse a previous pipeline to validate current commit. This action will also kill all currently running builds associated with the pull request. IMPORTANT NOTE: This is dangerous since lack of user care and validation can cause top of tree to break.

LinPoly avatar May 13 '25 11:05 LinPoly

/bot run

LinPoly avatar May 13 '25 11:05 LinPoly

PR_Github #5007 [ run ] triggered by Bot

tensorrt-cicd avatar May 13 '25 12:05 tensorrt-cicd

PR_Github #5007 [ run ] completed with state SUCCESS /LLM/main/L0_MergeRequest_PR pipeline #3640 completed with status: 'FAILURE'

tensorrt-cicd avatar May 13 '25 13:05 tensorrt-cicd

/bot run

LinPoly avatar May 14 '25 06:05 LinPoly

PR_Github #5125 [ run ] triggered by Bot

tensorrt-cicd avatar May 14 '25 07:05 tensorrt-cicd

PR_Github #5125 [ run ] completed with state SUCCESS /LLM/main/L0_MergeRequest_PR pipeline #3738 completed with status: 'FAILURE'

tensorrt-cicd avatar May 14 '25 10:05 tensorrt-cicd

/bot run

LinPoly avatar May 15 '25 09:05 LinPoly

PR_Github #5327 [ run ] triggered by Bot

tensorrt-cicd avatar May 15 '25 09:05 tensorrt-cicd

/bot kill

LinPoly avatar May 15 '25 09:05 LinPoly

PR_Github #5332 [ kill ] triggered by Bot

tensorrt-cicd avatar May 15 '25 10:05 tensorrt-cicd

PR_Github #5327 [ run ] completed with state ABORTED

tensorrt-cicd avatar May 15 '25 10:05 tensorrt-cicd

PR_Github #5332 [ kill ] completed with state SUCCESS Successfully killed previous jobs for commit 9a08a65

tensorrt-cicd avatar May 15 '25 10:05 tensorrt-cicd

/bot run

kaiyux avatar May 16 '25 05:05 kaiyux

PR_Github #5456 [ run ] triggered by Bot

tensorrt-cicd avatar May 16 '25 05:05 tensorrt-cicd

PR_Github #5456 [ run ] completed with state SUCCESS /LLM/main/L0_MergeRequest_PR pipeline #3981 completed with status: 'FAILURE'

tensorrt-cicd avatar May 16 '25 08:05 tensorrt-cicd

/bot run

LinPoly avatar May 16 '25 08:05 LinPoly

PR_Github #5490 [ run ] triggered by Bot

tensorrt-cicd avatar May 16 '25 08:05 tensorrt-cicd

PR_Github #5490 [ run ] completed with state SUCCESS /LLM/main/L0_MergeRequest_PR pipeline #4005 completed with status: 'FAILURE'

tensorrt-cicd avatar May 16 '25 20:05 tensorrt-cicd

/bot run

LinPoly avatar May 17 '25 05:05 LinPoly

PR_Github #5561 [ run ] triggered by Bot

tensorrt-cicd avatar May 17 '25 05:05 tensorrt-cicd

PR_Github #5561 [ run ] completed with state FAILURE /LLM/main/L0_MergeRequest_PR pipeline #4058 completed with status: 'FAILURE'

tensorrt-cicd avatar May 17 '25 05:05 tensorrt-cicd

/bot run

LinPoly avatar May 17 '25 05:05 LinPoly

PR_Github #5562 [ run ] triggered by Bot

tensorrt-cicd avatar May 17 '25 06:05 tensorrt-cicd

PR_Github #5562 [ run ] completed with state SUCCESS /LLM/main/L0_MergeRequest_PR pipeline #4059 completed with status: 'FAILURE'

tensorrt-cicd avatar May 17 '25 08:05 tensorrt-cicd

/bot run

LinPoly avatar May 18 '25 11:05 LinPoly

PR_Github #5614 [ run ] triggered by Bot

tensorrt-cicd avatar May 18 '25 11:05 tensorrt-cicd

PR_Github #5614 [ run ] completed with state SUCCESS /LLM/main/L0_MergeRequest_PR pipeline #4099 completed with status: 'SUCCESS'

tensorrt-cicd avatar May 18 '25 16:05 tensorrt-cicd