已合并
随机数算子资料更新 #4270
sikaiwei创建于 21 天前
随机数算子资料更新 #4270
已合并
Pull Request已成功合入, 合并人@CANN-robot
(感谢 sikaiwei 的贡献)atomgit-bot
21 天前 评论:
21 天前 评论:
变更摘要
本 PR 主要对随机数算子的相关文档和资料进行更新,涉及中英文用户手册、私有化方案文档、算子清单以及测试数据文件的调整。整体以删除冗余内容和精简描述为主(+18/-30 行),属于文档维护类变更。
主要改动
- 随机数算子文档内容修正:更新了
docs/docs/user-manual/resource-manager/random-number-generator.md和operator-random.md中的描述,以及对应的英文版(docs/docs/en/)和私有化方案文档(docs/docs/private-solution/)中的相关内容,保持多语言和多版本间的一致。 - 算子清单更新:
features/operators/random/manifests/operator_random_1_0_0.json中有 1 增 1 删的细微调整,可能涉及算子元数据或参数描述的修正。 - 测试数据精简:
ba173f44-5f3d-11f0-800c-52540045661f.json是变更量最大的文件(+8/-18 行),删除了较多测试数据内容,可能是清理过期或用例的用例。 - 文档站点配置微调:
docs/package.json和docs/.dumirc.ts有少量变动,属于文档工程配置的配套调整。


atomgit-bot
21 天前 评论:
21 天前 评论:
代码审查
审查总结
按优先级统计:
- P1(高概率运行时/用户可见问题):4 个 — 4 个文件中的
aclnn_return_code.md→aclnn返回码.md链接目标变更导致文档内链断裂 - P2(特定条件下触发的缺陷):1 个 —
aclnnInplaceUniformTensor.md中offsetTensor的 4 倍数约束被意外删除 - P3(低风险一致性问题):1 个 — 6 个文件中
<!-- npu="950" idX -->开标记被移除但闭标记残留,与同 PR 其余 4 个文件不一致
整体风险评估:中等风险。核心问题是 4 个文件中的 8 处文档链接断裂(P1),用户点击这些链接将遇到 404 错误。offsetTensor 约束丢失(P2)可能导致使用该接口的用户传递不合规的 offsetTensor 值。npu 标记不一致(P3)影响较低,但建议统一处理以避免文档构建工具异常。
逐文件审查确认:
| 文件 | 审查结果 |
|---|---|
random/dsa_random_normal/docs/aclnnInplaceNormal.md |
npu 标记残留(P3) |
random/dsa_random_normal/docs/aclnnInplaceNormalTensor.md |
链接断裂(P1)+ npu 标记残留(P3) |
random/dsa_random_uniform/docs/aclnnInplaceRandom.md |
无问题 |
random/dsa_random_uniform/docs/aclnnInplaceRandomTensor.md |
链接断裂(P1) |
random/dsa_random_uniform/docs/aclnnInplaceUniform.md |
链接断裂(P1) |
random/dsa_random_uniform/docs/aclnnInplaceUniformTensor.md |
链接断裂(P1)+ offsetTensor 约束丢失(P2) |
random/stateless_random_normal_v2/docs/aclnnNormalFloatFloat.md |
npu 标记残留(P3) |
random/stateless_random_normal_v2/docs/aclnnNormalFloatTensor.md |
npu 标记残留(P3) |
random/stateless_random_normal_v2/docs/aclnnNormalTensorFloat.md |
npu 标记残留(P3) |
random/stateless_random_normal_v2/docs/aclnnNormalTensorTensor.md |
npu 标记残留(P3) |
| 类型 | 数量 |
|---|---|
| 🔴 阻塞 | 4 |
| 🟡 建议 | 0 |
⛔ 需要修改


21 天前 添加了label:cann-cla/yes
CANN-robot
21 天前 评论:
21 天前 评论:
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 |
|---|---|---|
| */*/docs/acl*.md | ✅ zhangzijie, 陈娇 (2/2) | ✅ 陈娇 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
sikaiwei, thanks for your pull request. All authors of the commits have signed the CLA. 👍


此处折叠了47条消息 查看更多
zhangzijie
20 天前 评论:
20 天前 评论:
/approve


20 天前 添加了label:lgtm
20 天前 合入了pull request
描述
更新随机数算子的aclnn的资料
关联的Issue
测试
文档更新
更新了随机数算子的aclnn的md文件
类型标签