Skip to content

Conversation

mgoin
Copy link
Member

@mgoin mgoin commented Sep 10, 2025

Purpose

This log is unneeded as it is shown by default

(APIServer pid=3266965) INFO 09-10 14:38:34 [async_llm.py:173] Torch profiler disabled. AsyncLLM CPU traces will not be collected.

Test Plan

Test Result


Essential Elements of an Effective PR Description Checklist
  • The purpose of the PR, such as "Fix some issue (link existing issues this PR will resolve)".
  • The test plan, such as providing test command.
  • The test results, such as pasting the results comparison before and after, or e2e results
  • (Optional) The necessary documentation update, such as updating supported_models.md and examples for a new model.
  • (Optional) Release notes update. If your change is user facing, please update the release notes draft in the Google Doc.

Signed-off-by: mgoin <mgoin64@gmail.com>
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request removes an informational log message that indicates when the Torch profiler is disabled. The change is intended to reduce log verbosity, as the profiler being disabled is the default state. The modification is simple, safe, and aligns with the goal of improving the user experience by cleaning up the logs. I approve of this change.

@simon-mo simon-mo merged commit 9d2a446 into vllm-project:main Sep 12, 2025
15 checks passed
skyloevil pushed a commit to skyloevil/vllm that referenced this pull request Sep 13, 2025
dsxsteven pushed a commit to dsxsteven/vllm_splitPR that referenced this pull request Sep 15, 2025
bbartels pushed a commit to bbartels/vllm that referenced this pull request Sep 15, 2025
Signed-off-by: mgoin <mgoin64@gmail.com>
Signed-off-by: bbartels <benjamin@bartels.dev>
FeiDaLI pushed a commit to FeiDaLI/vllm that referenced this pull request Sep 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants