Skip to content

NewRLInfra-3003 add over-context-len prompt tokens into stat - #8111

Merged
rain7996 merged 2 commits into
PaddlePaddle:developfrom
rain7996:develop
Aug 24, 2026
Merged

NewRLInfra-3003 add over-context-len prompt tokens into stat#8111
rain7996 merged 2 commits into
PaddlePaddle:developfrom
rain7996:develop

Conversation

@rain7996

@rain7996 rain7996 commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

Motivation

Add over-context-len prompt tokens into stat in sglang benchmarks. Over-context-len prompt will be sglang-serving prefilled and get an emtpy response which is marked as a failed request. It leads to lower ITPS than the server logs in stat.

Modifications

Add the input-len of over-context-len prompts into stat.

Usage or Command

No additional command.

Accuracy Tests

Align ITPS with sglang server logs.

Checklist

  • [ x] Add at least a tag in the PR title.
    • Tag list: [[FDConfig],[APIServer],[Engine], [Scheduler], [PD Disaggregation], [Executor], [Graph Optimization], [Speculative Decoding], [RL], [Models], [Quantization], [Loader], [OP], [KVCache], [DataProcessor], [BugFix], [Docs], [CI], [Optimization], [Feature], [Benchmark], [Others], [XPU], [HPU], [GCU], [DCU], [Iluvatar], [Metax]]
    • You can add new tags based on the PR content, but the semantics must be clear.
  • [ x] Format your code, run pre-commit before commit.
  • Add unit tests. Please write the reason in this PR if no unit tests.
  • Provide accuracy results.
  • If the current PR is submitting to the release branch, make sure the PR has been submitted to the develop branch, then cherry-pick it to the release branch with the [Cherry-Pick] PR tag.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@rain7996
rain7996 merged commit 947f323 into PaddlePaddle:develop Aug 24, 2026
41 of 43 checks passed
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.

3 participants