Skip to content

Adapt the latest sglang integration test Engine kwargs.#1964

Merged
changwangss merged 2 commits into
mainfrom
wangchang/sglang
Jun 29, 2026
Merged

Adapt the latest sglang integration test Engine kwargs.#1964
changwangss merged 2 commits into
mainfrom
wangchang/sglang

Conversation

@changwangss

@changwangss changwangss commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Description

Root cause: sglang 0.5.14 removed legacy Engine/ServerArgs kwargs (disable_piecewise_cuda_graph, cuda_graph_bs) in favor of the new CUDA graph config fields, so the old test arguments became invalid.

I improved ut as following.

  • Fix sglang integration test by adapting Engine kwargs to ServerArgs changes in newer sglang.
  • Use legacy args when available, otherwise fall back to new args (disable_cuda_graph / cuda_graph_bs_decode).
  • Keeps backward compatibility and restores test_ar_format_sglang on sglang 0.5.14.

Type of Change

Bug fix

#1961

Fixes or relates to #

Checklist Before Submitting

  • My code has been tested locally.
  • Documentation has been updated as needed.
  • New or updated tests are included where applicable.
  • The CUDA CI has passed. You can trigger it by commenting /azp run Unit-Test-CUDA-AutoRound.

changwangss and others added 2 commits June 29, 2026 08:30
Signed-off-by: changwangss <chang1.wang@intel.com>
@changwangss changwangss merged commit 4924454 into main Jun 29, 2026
17 checks passed
@changwangss changwangss deleted the wangchang/sglang branch June 29, 2026 13:08
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.

2 participants