已合并
modified md files #1118
gitee-duhuiping创建于 7月13日
modified md files #1118
已合并
Pull Request已成功合入, 合并人@CANN-robot
(感谢 gitee-duhuiping 的贡献)7月13日 创建了 pull request,commit 526489d2
atomgit-bot
7月13日 评论:
7月13日 评论:
atomgit-bot
7月13日 评论:
7月13日 评论:
代码审查
审查总结
本次 diff 涉及 38 个 markdown 文档的修改,主要是公式修正、笔误修复和格式整理。经过逐文件审查,发现 2 个问题:
| 优先级 | 数量 | 说明 |
|---|---|---|
| P2 | 1 | aclnnUpsampleNearestExact1dBackward.md 中 scalesL 公式被错误反转 |
| P3 | 1 | Exact2d/Exact3d 反向 README 中的 scales 公式未与详细文档同步修正 |
各文件审查结果
- docs/zh/op_api_list.md — API 列表反向传播引用修正,无问题
- image/grid_sample/README.md — 3D/2D 坐标映射修正(D/H/W↔x/y/z)、双线性求和上下界修正、最近邻输出变量名修正,无问题
- image/grid_sample/docs/aclnnGridSampler2D.md — 同上坐标映射及双线性求和修正,无问题
- image/grid_sample/docs/aclnnGridSampler3D.md — 同上,无问题
- image/grid_sampler2_d_grad/README.md — 双三次公式中
x→tx修正、tx/ty定义前移,无问题 - image/grid_sampler2_d_grad/docs/aclnnGridSampler2DBackward.md — 同上,无问题
- image/grid_sampler3_d_grad/README.md —
giy_mult→giz_mult修正,无问题 - image/grid_sampler3_d_grad/docs/aclnnGridSampler3DBackward.md — 同上 + 格式修正 +
paddingMode拼写修正,无问题 - image/rasterizer/docs/aclnnRasterizer.md — 缩进/格式/步骤编号修正,无问题
- image/resize_bilinear_v2/docs/aclnnResize.md — shape 约束补充 "size" 字样,无问题
- image/resize_nearest_neighbor_v2/docs/aclnnUpsampleNearest2d.md — 60G→60GB、
sizeof(float)→sizeof(dtype)+ dtype 说明,无问题 - image/resize_upsample_trilinear/docs/aclnnUpsampleTrilinear3d.md — 同上 + "输出shape"→"输入shape" 约束修正,无问题
- image/upsample_bicubic2d/docs/aclnnUpsampleBicubic2d.md — 同上类修正,无问题
- image/upsample_bicubic2d_aa/docs/aclnnUpsampleBicubic2dAA.md — True→true + 同上类修正,无问题
- image/upsample_bicubic2d_aa_grad/docs/aclnnUpsampleBicubic2dAAGrad.md — 同上 + 描述修正(放大→缩小),无问题
- image/upsample_bilinear2d/README.md —
alignCorners=true→false(边对齐描述修正),无问题 - image/upsample_bilinear2d/docs/aclnnUpsampleBilinear2d.md — 同上,无问题
- image/upsample_bilinear2d_aa_backward/docs/aclnnUpsampleBilinear2dAABackward.md — 同上类修正,无问题
- image/upsample_bilinear2d_grad/docs/aclnnUpsampleBilinear2dBackwardV2.md — rowspan 12→10 修正(确认为 10 行)+ dtype 修正,无问题
- image/upsample_bilinear2d_grad/docs/aclnnUpsampleLinear1dBackward.md — 描述修正 + dtype 修正,无问题
- image/upsample_linear1d/README.md —
scale_h→scale统一命名 + 尾随空格清理,无问题 - image/upsample_linear1d/docs/aclnnUpsampleLinear1d.md — True→true + 同上,无问题
- image/upsample_nearest/docs/aclnnUpsampleNearestExact1d.md — dtype 说明补充,无问题
- image/upsample_nearest/docs/aclnnUpsampleNearestExact2d.md — 同上,无问题
- image/upsample_nearest2d_grad/README.md —
srcHUp→srcWUp修正,无问题 - image/upsample_nearest2d_grad/docs/aclnnUpsampleNearest1dBackward.md — 仅格式清理,scales 未变(保持正确的
inputSize/outputSize),无问题 - image/upsample_nearest2d_grad/docs/aclnnUpsampleNearest2dBackward.md — 仅格式清理,无问题
- image/upsample_nearest3d/README.md — 公式格式整理(合并为一行),无问题
- image/upsample_nearest3d/docs/aclnnUpsampleNearest1dV2.md —
\quad格式修正,无问题 - image/upsample_nearest3d/docs/aclnnUpsampleNearest2dV2.md — 同上,无问题
- image/upsample_nearest3d/docs/aclnnUpsampleNearest3d.md — 同上,无问题
- image/upsample_nearest3d_grad/docs/aclnnUpsampleNearest3dBackward.md — scales 反转(对 3D 正确)+ dtype 修正,无问题
- image/upsample_nearest_exact2d_grad/README.md —
srcWUp修正,但 scales 未同步 → P3 - image/upsample_nearest_exact2d_grad/docs/aclnnUpsampleNearestExact1dBackward.md — scales 反转错误 → P2
- image/upsample_nearest_exact2d_grad/docs/aclnnUpsampleNearestExact2dBackward.md — scales 反转(对 Exact2d 正确)+
srcWUp修正,无问题 - image/upsample_nearest_exact3d_grad/README.md —
srcWUp修正,但 scales 未同步(同 P3),无独立问题 - image/upsample_nearest_exact3d_grad/docs/aclnnUpsampleNearestExact3dBackward.md — scales 反转(对 Exact3d 正确)+
srcWUp修正,无问题 - image/upsample_trilinear3d_backward/docs/aclnnUpsampleTrilinear3dBackward.md — dtype 修正,无问题
整体风险评估
中等风险。 P2 问题涉及 aclnnUpsampleNearestExact1dBackward 反向公式的文档错误,若开发者参照该文档实现或验证反向传播逻辑,将得到错误的梯度计算结果。P3 问题为文档内部不一致,影响较低。其余 36 个文件的修改均为正确的文档修正,不引入新问题。
| 类型 | 数量 |
|---|---|
| 🔴 阻塞 | 0 |
| 🟡 建议 | 1 |
💬 仅评论


7月13日 添加了label:cann-cla/yes
CANN-robot
7月13日 评论:
7月13日 评论:
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 |
|---|---|---|
| */*/README.md | ✅ 周文芳, 陈娇, 吴瑶, 刘伟, 周奇龙 (5/2) | ✅ 周文芳, 陈娇 (2/1) |
| */*/docs/acl*.md | ✅ 周文芳, 周奇龙, 吴瑶, 刘伟, 陈娇 (5/2) | ✅ 陈娇, 周文芳 (2/1) |
| repo-cann/ops-cv | ✅ 刘伟, 周奇龙 (2/2) | ✅ 刘伟, 周奇龙 (2/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
gitee-duhuiping, thanks for your pull request. All authors of the commits have signed the CLA. 👍


此处折叠了89条消息 查看更多
CANN-robot
7月15日 评论:
7月15日 评论:
The following users do not have permission to comment /lgtm or /approve on any module in this PR:
宋恺


7月15日 添加了label:lgtm
7月15日 解决了最后一个问题
7月15日 合入了pull request
描述
modified md files
关联的Issue
NA
测试
Only involves updating the MD document description, does not involve testing.
文档更新
md files
类型标签