已合并
[docs]刷新D2D特性说明文档 #627
yilunh创建于 7月28日
[docs]刷新D2D特性说明文档 #627
已合并
yilunh创建于 7月28日
yilunh
7月28日

1. 合入背景

刷新D2D特性说明文档 fixes #370

2. 修改内容

1、刷新配置int8_cache配置建议值,默认使用no cache直通
2、MTP叠加时建议端口预留,防止端口非法,elastic server无法启动
3、刷新支持的模型
4、刷新文档位置至docs/user_guide/features

3. 资料变更

4. 接口变更

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

5. 测试结果

6. CheckList

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

[ ] 代码注释完备

[ ] 正确记录维测日志

[ ] 是否有UT用例

[ ] 若涉及多线程场景,考虑了并发场景,不存在死锁问题

likedislike
Pull Request已成功合入, 合并人@wangyang
(感谢 yilunh 的贡献)
atomgit-bot
atomgit-bot
7月28日 评论:

变更摘要

本次 PR 主要对 D2D 特性说明文档进行刷新更新,属于纯文档类变更。核心改动包括:调整 int8_cache 配置的建议值为 no cache 直通模式、新增 MTP 叠加场景下的端口预留建议以避免 elastic server 启动失败、以及更新当前支持的模型列表。


主要改动

  • int8_cache 配置建议值更新:将配置建议从原有值改为默认使用 no cache 直通模式,以反映最新的推荐实践。
- **MTP 叠加端口预留说明**:新增在 MTP 叠加部署时建议预留端口的说明,防止因端口冲突导致 elastic server 无法正常启动。
  • 支持模型列表刷新:更新文档中 D2D 特性所支持的模型清单,确保文档与当前实际支持的模型保持一致。
likedislike
atomgit-bot
atomgit-bot
7月28日 评论:

代码审查

现在我已经审查完毕。以下是总结。


审查总结

审查文件docs/zh/user_guide/features/startup_acceleration.md — 发现 3 个问题。

优先级 数量 说明
P2 1 Qwen3 模型参数规模标注错误(30B → 235B),可能误导用户选错配置
P3 2 术语不一致(RANK / device_rank);int8_cache 合法取值丢失

整体风险判断:低风险。此 diff 为文档刷新,主要是刷新配置建议值、MTP 端口预留建议、已测试模型列表及 ISSUE 链接。P2 的模型名标注错误是唯一值得修复的实质性问题——它可能使用户将 300 亿参数模型的配置误用于 2350 亿参数场景或反之。其余 P3 项为文档 polish,可在后续一并修正。

类型 数量
🔴 阻塞 0
🟡 建议 1

💬 仅评论

likedislike
ascend-robotascend-robot成员
7月28日 添加了label:ascend-cla/yes
ascend-robot
ascend-robot成员
7月28日 评论:

CLA Signature Pass

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

likedislike
ascend-robot
ascend-robot成员
7月28日 评论:

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-Motor 吕有辉, 於泽辰 (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

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

likedislike
此处折叠了87条消息 查看更多
jason lyujason lyu成员
20 天前 解决了最后一个问题
jason lyu
jason lyu成员
20 天前 评论:

/approve

likedislike
ascend-robotascend-robot成员
20 天前 添加了label:approvedlgtm
wangyangwangyang成员
20 天前 关闭了关联的issue
wangyangwangyang成员
20 天前 合入了pull request