已合并

修复动态chunksize导致的bfcl精度问题 #908

lijiaqi139创建于 4月16日从lijiaqi139/MindIE-LLM： fix_chunk_dynamic_chunksize 合入到 Ascend/MindIE-LLM： dev

修复动态chunksize导致的bfcl精度问题 #908

已合并

lijiaqi139创建于 4月16日

从lijiaqi139/MindIE-LLM fix_chunk_dynamic_chunksize合入到Ascend/MindIE-LLM dev

lijiaqi139

4月16日

合入背景

Fixes #524

修改内容

请描述修改内容的具体实现，涉及哪些组件之间进行交互，可以用1、2、3、...进行罗列。
如果是需求或者重构类的PR，需要补充详细设计文档（说明上下游组件关系、时序图、类图、DFX能力等内容）。

资料变更

请确认是否涉及资料变更。如涉及，需要在PR中体现，并简要说明修改内容。如不涉及，需填写“不涉及”。

接口变更

请确认是否涉及跨代码仓或者客户面可见的接口变更。如涉及，需要详细说明接口以及对应的变更内容，同时需要在资料中体现。如不涉及，需填写“不涉及”。

测试结果

请说明测试场景，测试方法以及测试结果。
测试用例设计时需考虑硬件、部署方式、功能、性能、精度、显存等维度。

CheckList

PR提交人对以下CheckList自检项进行全量自检，自检通过或不涉及，均修改 [ ] 为 [x]。

代码注释完备
正确记录错误日志
进行了返回值校验 (禁止使用void屏蔽安全函数、自研函数返回值；考虑接口的异常场景；调用底层组件接口时，需要进行返回值校验)
进行了空指针校验
若存在资源申请，使用后资源被正确的释放了
若涉及多线程场景，考虑了并发场景，不存在死锁问题
按照代码仓中提供的格式模板，使用clang-format工具格式化代码
符合Ascend社区的编码规范。C++ 语言编程指导 | C++ 语言安全编程指导

Pull Request已成功合入, 合并人@ascend-robot

（感谢 lijiaqi139 的贡献）

ascend-robot成员

4月16日评论：

Thanks for your pull-request.
The full list of commands accepted by me can be found at here。
You can get sig-info at here

PR Approval Progress

✅ Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module	lgtm status	approve status
repo-Ascend/MindIE-LLM	✅ hw-zhoutianyang, 纪涛 (2/2)	✅ 纪涛 (1/1)

💡 Tip:

Committer can comment /approve or /lgtm

Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

lijiaqi139, thanks for your pull request. All authors of the commits have signed the CLA. 👍

ascend-robot成员

4月16日添加了label：ascend-cla/yes

ascend-robot成员

4月16日添加了label：needs-issue

ascend-robot成员

4月16日评论：

Linking Issue Notice

@lijiaqi139 , the pull request must be linked to at least one issue.
If an issue has already been linked, but the needs-issue label remains, you can remove the label by commenting /check-issue .

lijiaqi139

4月16日评论：

compile

ascend-robot成员

4月16日评论：

问题/功能描述

修复调度预算计算中，当未配置固定预填充分块大小时，预填充槽位计算逻辑缺失的问题。原代码在 prefillChunkSize 为0时，缺少明确的返回值，导致后续的“动态切分长度”逻辑可能被错误执行，影响调度决策的准确性。

修改方案描述

在 GetPrefillSlots 方法的“固定切分长度”判断分支中，当 prefillChunkSize 为0时，明确返回 maxPrefillTokens 作为预填充槽位值。该修改确保了在未启用固定分块策略时，调度器能使用配置的最大预填充令牌数作为预算依据，使逻辑更加完整和清晰，避免了潜在的逻辑错误。

ascend-robot成员

4月16日评论：

✅ 流水线任务触发成功(流水线编号: 4971)，正在执行，请稍候。任务链接点此跳转 ...

ascend-robot成员

4月16日添加了label：ci-pipeline-running

ascend-robot成员

4月16日评论：

Ascend docs pipeline is running...

ascend-robot成员

4月16日添加了label：docs-ci-pipeline-running

ascend-robot成员

4月16日评论：

Ascend docs pipeline is running...

ascend-robot成员

4月16日评论：

✅ 跳过 docs ci 检查，没有需要检查的文档文件

ascend-robot成员

4月16日删除了label：docs-ci-pipeline-running

ascend-robot成员

4月16日添加了label：docs-ci-pipeline-success

ascend-robot成员

4月16日评论：

✅ 跳过 docs ci 检查，没有需要检查的文档文件

L lijiaqi139

4月16日修改了pull request 的描述

ascend-robot成员

4月16日删除了label：needs-issue

ascend-robot成员

4月16日删除了label：ci-pipeline-running

ascend-robot成员

4月16日添加了label：ci-pipeline-passed

ascend-robot成员

4月16日评论：

流水线 PR-pipeline_MindIE-LLM_gitcode#4971 已完成

阶段	任务名	状态	详情
编译构建	Build_ops_x86_A3	✅	>>>
	Build_ops_arm_A3	✅	>>>
	Build_ops_x86_A2	✅	>>>
	Build_ops_arm_A2	✅	>>>
	Build_linux_x86_abi1_atb	✅	>>>
	Build_linux_x86_abi1_llm	✅	>>>
	Build_linux_arm_abi1_atb	✅	>>>
	Build_linux_arm_abi1_llm	✅	>>>
	Build_linux_arm_abi0_atb	✅	>>>
	Build_linux_arm_abi0_llm	✅	>>>
恶意代码检查	Antipoison	✅	>>>
编码安全与规范检查	pre-commit	✅	>>>
开源片段检查	SCA	✅	>>>
开发者测试	UT_linux_arm_mindie-llm	✅	>>>
开发者测试	UT_linux_arm_atb-models	✅	>>>
流水线	PR-pipeline_MindIE-LLM_gitcode	✅	>>>

纪涛成员

4月17日评论：

/approve

ascend-robot成员

4月17日添加了label：approved

纪涛成员

4月17日评论：

/merge

ascend-robot成员

4月17日添加了label：keeper_approved

hw-zhoutianyang

4月17日评论：

/lgtm

ascend-robot成员

4月17日添加了label：lgtm

ascend-robot成员

4月17日评论：

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: 纪涛.
Reviewers who wrote a comment of /lgtm are: 纪涛, hw-zhoutianyang.

ascend-robot成员

4月17日合入了pull request