Skip to content

chore: Deprecate autopp. - #4471

Merged
yuxianq merged 2 commits into
NVIDIA:release/0.20from
yuxianq:deprecate-autopp
May 21, 2025
Merged

chore: Deprecate autopp.#4471
yuxianq merged 2 commits into
NVIDIA:release/0.20from
yuxianq:deprecate-autopp

Conversation

@yuxianq

@yuxianq yuxianq commented May 20, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

@yuxianq

yuxianq commented May 20, 2025

Copy link
Copy Markdown
Collaborator Author

/bot run

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #5824 [ run ] triggered by Bot

Signed-off-by: Yuxian Qiu <142763828+yuxianq@users.noreply.github.com>
@yuxianq
yuxianq force-pushed the deprecate-autopp branch from e46d596 to 1196bc0 Compare May 20, 2025 05:25
@yuxianq
yuxianq changed the base branch from main to release/0.20 May 20, 2025 05:25
@yuxianq

yuxianq commented May 20, 2025

Copy link
Copy Markdown
Collaborator Author

/bot run

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #5826 [ run ] triggered by Bot

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #5824 [ run ] completed with state ABORTED

@Superjomn Superjomn left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

Are there any other examples/docs that we need to add the deprecation warning as well?

Signed-off-by: Yuxian Qiu <142763828+yuxianq@users.noreply.github.com>
@yuxianq
yuxianq requested a review from a team as a code owner May 21, 2025 04:17
@yuxianq

yuxianq commented May 21, 2025

Copy link
Copy Markdown
Collaborator Author

Are there any other examples/docs that we need to add the deprecation warning as well?

@Superjomn I also add deprecation warning to docs. As for examples, they will give the deprecation warning once the user run it with autopp, so we don't have to modify them.

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #5826 [ run ] completed with state ABORTED

@yuxianq

yuxianq commented May 21, 2025

Copy link
Copy Markdown
Collaborator Author

/bot skip --comment "This PR only contains deprecation warning, all single-gpu tests and most multi-gpu tests are passed."

@yuxianq
yuxianq enabled auto-merge (squash) May 21, 2025 05:38
@yuxianq
yuxianq requested a review from litaotju May 21, 2025 05:38
@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #5967 [ skip ] triggered by Bot

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #5967 [ skip ] completed with state SUCCESS
Skipping testing for commit 4e6515e

@yuxianq
yuxianq merged commit ff0f37b into NVIDIA:release/0.20 May 21, 2025
amirkl94 pushed a commit to amirkl94/TensorRT-LLM that referenced this pull request May 28, 2025
Signed-off-by: Yuxian Qiu <142763828+yuxianq@users.noreply.github.com>
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