Skip to content

Conversation

@zhoutianzi666
Copy link
Collaborator

@zhoutianzi666 zhoutianzi666 commented Nov 19, 2025

Motivation

删除get_block_shape方法中的冗余代码,使得代码更加简洁,并添加decoder_batch_ele_num替代常量。

Modifications

根据输入计算decoder_batch_ele_num变量代替原本的固定常量。

Usage or Command

优化代码写法

Accuracy Tests

验证通过

Checklist

  • 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.
  • 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.

@paddle-bot
Copy link

paddle-bot bot commented Nov 19, 2025

Thanks for your contribution!

@zhoutianzi666 zhoutianzi666 changed the title remove param get_block_shape_and_split_kv_block remove param Nov 19, 2025
@zhoutianzi666 zhoutianzi666 changed the title get_block_shape_and_split_kv_block remove param get_block_shape_and_split_kv_block clean code Nov 19, 2025
chang-wenbin
chang-wenbin previously approved these changes Nov 19, 2025
@zhoutianzi666 zhoutianzi666 changed the title get_block_shape_and_split_kv_block clean code [Others]get_block_shape_and_split_kv_block clean code Nov 20, 2025
@zhoutianzi666 zhoutianzi666 merged commit 6fa3410 into PaddlePaddle:develop Nov 20, 2025
14 of 17 checks passed
@zhoutianzi666 zhoutianzi666 deleted the remove_code branch November 20, 2025 13:31
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