Skip to content

Allow custom policy for adaptive speculative decoding - #37274

Merged
merrymercy merged 2 commits into
sgl-project:mainfrom
jasonjk-park:jasonjk/custom-adaptive-policy
Sep 2, 2026
Merged

merrymercy merged 2 commits into
sgl-project:mainfrom
jasonjk-park:jasonjk/custom-adaptive-policy

Conversation

@jasonjk-park

@jasonjk-park jasonjk-park commented Aug 31, 2026 •

Copy link
Copy Markdown
Contributor

Motivation

Allow custom speculative algorithm to inject custom policy for adaptive speculative decoding

Modifications

(1) Add AdaptiveSpecPolicy that one can refer to for custom policy
(2) Always pass in AdaptiveSpecPolicy rather than config

Accuracy Tests

N/A

Speed Tests and Profiling

N/A

Checklist

Review and Merge Process

  1. Ping Merge Oncalls to start the process. See the PR Merge Process.
  2. Get approvals from CODEOWNERS and other reviewers.
  3. Trigger CI tests with comments or contact authorized users to do so.
    • Common commands include /tag-and-rerun-ci, /tag-run-ci-label, /rerun-failed-ci
  4. After green CI and required approvals, ask Merge Oncalls or people with Write permission to merge the PR.

CI States

Latest PR Test (Base): ⏳ Run #33531992306
Latest PR Test (Extra): ❌ Run #33531992053
Latest PR Test (AMD ROCm 7.2): ⏳ Run #33531992320

@jasonjk-park

jasonjk-park commented Sep 1, 2026 •

Copy link
Copy Markdown
Contributor Author

/tag-and-rerun-ci retry

@github-actions github-actions Bot added the run-ci CI: run the baseline test suite on this PR label Sep 1, 2026
@jasonjk-park

jasonjk-park commented Sep 1, 2026 •

Copy link
Copy Markdown
Contributor Author

/rerun-failed-ci retry

@merrymercy
merrymercy merged commit 83bd2c4 into sgl-project:main Sep 2, 2026
477 of 580 checks passed
StevenChenSE pushed a commit to StevenChenSE/sglang that referenced this pull request Sep 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-ci CI: run the baseline test suite on this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants