Skip to content

Enable chunked prefill by default for supported execution providers - #939

Merged
baijumeswani merged 2 commits into
mainfrom
baijumeswani/chunked-prefill
Aug 4, 2026
Merged

Enable chunked prefill by default for supported execution providers#939
baijumeswani merged 2 commits into
mainfrom
baijumeswani/chunked-prefill

Conversation

@baijumeswani

Copy link
Copy Markdown
Collaborator

Sets the ORT GenAI chunk_size search option to  2048  by default for CPU, CUDA, Nv TensorRT RTX, and WebGPU.

Existing positive chunk_size values are preserved. Missing or zero values use the new default, while ORT GenAI determines whether the model supports chunked prefill. Tests cover provider selection and override behavior.

Copilot AI review requested due to automatic review settings July 31, 2026 17:43
@vercel

vercel Bot commented Jul 31, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
foundry-local Ready Ready Preview Aug 4, 2026 7:57pm

Request Review

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Enables a default chunked-prefill size for selected execution providers.

Changes:

  • Adds CPU provider-name recognition.
  • Passes execution-provider context into search-option handling.
  • Adds tests for defaults, overrides, and provider resolution.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
execution_provider_test.cc Tests CPU provider aliases.
search_options_test.cc Tests chunk-size behavior.
search_options.h Extends the search-options interface.
search_options.cc Applies provider-specific chunk defaults.
onnx_chat_generator.cc Supplies the model execution provider.
execution_provider.h Recognizes CPU provider names.

Comment thread sdk_v2/cpp/src/inferencing/generative/chat/search_options.cc
@baijumeswani
baijumeswani changed the base branch from main to baijumeswani/upgrade-packages August 3, 2026 16:34
@baijumeswani

Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
2 pipeline(s) were filtered out due to trigger conditions.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated no new comments.

xiaoyu-work
xiaoyu-work previously approved these changes Aug 3, 2026
@baijumeswani

Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
2 pipeline(s) were filtered out due to trigger conditions.

bmehta001
bmehta001 previously approved these changes Aug 3, 2026
@baijumeswani
baijumeswani force-pushed the baijumeswani/upgrade-packages branch from ec9e87e to e8e163d Compare August 4, 2026 00:46
@baijumeswani
baijumeswani force-pushed the baijumeswani/chunked-prefill branch from d51cee3 to 39a9742 Compare August 4, 2026 00:46
@baijumeswani
baijumeswani force-pushed the baijumeswani/upgrade-packages branch from 11f6146 to 4d37878 Compare August 4, 2026 15:06
@baijumeswani
baijumeswani force-pushed the baijumeswani/chunked-prefill branch from 39a9742 to 58b827b Compare August 4, 2026 15:06
@baijumeswani
baijumeswani force-pushed the baijumeswani/chunked-prefill branch from 58b827b to 31d85c5 Compare August 4, 2026 16:54
Base automatically changed from baijumeswani/upgrade-packages to main August 4, 2026 18:55
@baijumeswani
baijumeswani force-pushed the baijumeswani/chunked-prefill branch from 31d85c5 to f7b39c2 Compare August 4, 2026 18:59
bmehta001
bmehta001 previously approved these changes Aug 4, 2026
@baijumeswani
baijumeswani enabled auto-merge (squash) August 4, 2026 19:10
@baijumeswani
baijumeswani merged commit 7e46075 into main Aug 4, 2026
50 checks passed
@baijumeswani
baijumeswani deleted the baijumeswani/chunked-prefill branch August 4, 2026 21:41
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