Skip to content

Ensure the pipe benchmark uses kwargs when calling pipe.stream_infer#4312

Merged
lvhan028 merged 1 commit intoInternLM:mainfrom
lvhan028:fix-benchmark-pipeline
Feb 2, 2026
Merged

Ensure the pipe benchmark uses kwargs when calling pipe.stream_infer#4312
lvhan028 merged 1 commit intoInternLM:mainfrom
lvhan028:fix-benchmark-pipeline

Conversation

@lvhan028
Copy link
Copy Markdown
Collaborator

@lvhan028 lvhan028 commented Feb 2, 2026

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the pipeline benchmark to call Pipeline.stream_infer with the correct gen_config keyword argument, ensuring generation configs aren’t misinterpreted as sessions.

Changes:

  • Pass gen_config=gen_configs (kwarg) to pipe.stream_infer in the streaming benchmark path.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Collaborator

@zhulinJulia24 zhulinJulia24 left a comment

Choose a reason for hiding this comment

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

LGTM

@lvhan028 lvhan028 merged commit 5cd82ae into InternLM:main Feb 2, 2026
8 of 9 checks passed
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.

3 participants